fix(nginx): fix broken redirects and multi-hop chains#2313
fix(nginx): fix broken redirects and multi-hop chains#2313
Conversation
TC-MO
commented
Mar 5, 2026
TC-MO
commented
- Fix 27 redirect destinations pointing to 404 pages (monitoring sub-pages, proxy examples, get-most-of-actors sub-pages, testing- and-maintenance, analyze-pages-and-fix-errors, scrape-with-sitemaps)
- Remove 2 unreachable duplicate rewrite rules (same source pattern appearing twice with different destinations — first match always won)
- Collapse ~80 multi-hop redirect chains into single hops: access- rights → collaboration, actors sub-pages, tutorials, development section restructure, data-collection → scraping-basics-javascript
- Fix 27 redirect destinations pointing to 404 pages (monitoring sub-pages, proxy examples, get-most-of-actors sub-pages, testing- and-maintenance, analyze-pages-and-fix-errors, scrape-with-sitemaps) - Remove 2 unreachable duplicate rewrite rules (same source pattern appearing twice with different destinations — first match always won) - Collapse ~80 multi-hop redirect chains into single hops: access- rights → collaboration, actors sub-pages, tutorials, development section restructure, data-collection → scraping-basics-javascript Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
TC-MO
added
documentation
|
Preview for this PR was built for commit |
danpoletaev
force-pushed
the
nginx-simplification
branch
from
b3f86f9 to
e96c70a
Compare
|
Preview for this PR was built for commit |
metalwarrior665
force-pushed
the
nginx-simplification
branch
from
e96c70a to
b3f86f9
Compare
|
Preview for this PR was built for commit |
|
@B4nan I guess the branch has conflicts now from your last PR but other than that, is there any way apart from YOLO'ing it to test those changes? |
|
I don't think those conflicts are from my PR (at least if we are talking about those from today), I haven't touched the redirects. Anyway, to truly test this, you'd have to run the whole thing locally, which is surely a bit too much. |
|
Fair enough, I'll resolve the conflicts and if I could get at least sanity check for that after I would greatly appreciate it |
|
Yeah, let's just not merge it late afternoon and leave that for morning instead. |
Resolve conflict in nginx.conf: keep branch's chain-collapsed destinations for get-most-of-actors redirects (actor-marketing-playbook), but use master's monetization destination (/platform/actors/publishing/monetize) from PR #2312. All master additions preserved (security backslash block, client JS/Python restructuring redirects). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Preview for this PR was built for commit |
3 participants
