-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tracker: ab180 #272
tracker: ab180 #272
Conversation
Did you mean AdguardTeam/AdguardFilters#129318? |
It seems they're using |
Your previous contribution to adguard filters actually does the same thing compared to this. |
Your suggestion resolves my concern on possible site breakages by blocking at domain level. Additionally, I confirmed the resource is not used on internal dashboard via business account including personal one. |
bedbb53
to
1d9366b
Compare
I rebased your branch with |
I confirmed your changes. |
Thanks! |
Why? The topic of the contribution to AdguardFilters and this PR is same. |
I don't understand what's your purpose to say like that. I mean the final outcome made by both two of them are same. |
Review required
A mitigation for Airbridge product; specially in deep-linking web to app integration and data collection.
Live sample:
https://www.sirloin.co.kr/
URL schemas:
$version
is the integration version between the data api and SDK$appDirectory
is namespace identifier of AirBridge customer consisted of commonly used characters in HTTP URL; commonly viewed as application name (e.g.my-application
)I suggest adding
$xhr
and$3p
if possible. However, I'm not sure about the status of sugar syntax support in YousList since the use of sugar syntax is literally low. In current version, I removed all sugars.Checks