Skip to content

Fix the description of the $urltransform modifier and update the tracker bypass example - #887

Open
ngorskikh wants to merge 2 commits into
masterfrom
feature/urltransform_clarifications
Open

Fix the description of the $urltransform modifier and update the tracker bypass example#887
ngorskikh wants to merge 2 commits into
masterfrom
feature/urltransform_clarifications

Conversation

@ngorskikh

Copy link
Copy Markdown
Member

Subj.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the documentation for the $urltransform modifier to reflect its transformation pipeline behavior and refreshes the affiliate/tracker bypass examples accordingly.

Changes:

  • Clarifies that $urltransform applies a series of transformations rather than a single regex replacement.
  • уточняет (documents) execution order when multiple $urltransform rules match: applied sequentially in lexicographical order.
  • Replaces the tracker-bypass example with modern pct/b64 transformation-based rules and adds a note about “Changing the origin”.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions

Copy link
Copy Markdown

Preview was deployed to: https://pull-request-887.kb-adg.pages.dev/

@Alex-302

Alex-302 commented Aug 3, 2026

Copy link
Copy Markdown
Member

@el-termikael ping

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

Preview was deployed to: https://pull-request-887.kb-adg.pages.dev/

`https://www.somestore.com/?ref=ref-123`. The final substitution removes the `ref` tracking parameter.

:::caution Changing the origin
Note that in both rules, the `$urltransform` value starts with `/^http`, so the full request URL is transformed. Without this prefix, only the path and query parts of the URL can be transformed.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Note that in both rules, the `$urltransform` value starts with `/^http`, so the full request URL is transformed. Without this prefix, only the path and query parts of the URL can be transformed.
Note that in both rules, the `$urltransform` value starts with `/^http`, so the full request URL is transformed. Without this prefix, only the path and query parts of the URL can be transformed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants