Skip to content

add "kc" and "kac" to ignore params - #77

Merged
ladamski merged 1 commit into
mainfrom
la/extend-ignore-params
Jun 9, 2026
Merged

add "kc" and "kac" to ignore params#77
ladamski merged 1 commit into
mainfrom
la/extend-ignore-params

Conversation

@ladamski

@ladamski ladamski commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

https://app.asana.com/1/137249556945/project/1209805270658160/task/1214934152266987?focus=true

Looks like it can occasionally occur for non-Windows pixels (which makes sense if via proxy injection)


Note

Low Risk
Config-only additions to ignore_params.json with no runtime logic changes; only affects which URL params pass pixel definition validation.

Overview
Extends the global pixel ignore params list so validation no longer treats kc and kac as unknown query parameters when third-party SERP proxies inject them (same treatment as existing kp / p entries, with allowed value -1).

This should reduce false validation noise on non-Windows pixels when those params appear in URLs.

Reviewed by Cursor Bugbot for commit 526aee6. Bugbot is set up for automated code reviews on this repo. Configure here.

@ladamski
ladamski requested a review from nshuba June 8, 2026 11:37
@ladamski
ladamski merged commit fb38d2a into main Jun 9, 2026
3 checks passed
@ladamski
ladamski deleted the la/extend-ignore-params branch June 9, 2026 23:49
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.

2 participants