Skip to content
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

Fix HTML content detection #855

Closed
Alex-302 opened this issue Feb 10, 2016 · 2 comments
Closed

Fix HTML content detection #855

Alex-302 opened this issue Feb 10, 2016 · 2 comments
Assignees
Labels
Milestone

Comments

@Alex-302
Copy link
Member

http://www.cowcotland.com/scripjs.php?files=jquery,jquery.easing.1.3

сontent-type text/html, but contains only js, without valid html tags
https://gist.github.com/Alex-302/8aa382fb3324da5349cd

https://www.minnetonkamoccasin.com/colorswatch/json/pull/

response: https://gist.github.com/Alex-302/55ba0ca087a195dac6c3

https://cupsservice.com/Double+Wall+Paper+Cups

response:
https://gist.github.com/Alex-302/653822881147a8b3c0cc

  • origin.com
    injects to js(content-type: text/html)
https://eaassets01-a.akamaihd.net/origin-com-store-webassets/1.0.0.410277_856d9a751163136d6a24075ada5f26dd2db13523050d56cab862536e02d823e0/js/ea5d46c.js

response:
https://gist.github.com/Alex-302/54524224b28dae2e241e

https://www.watersidenursery.co.uk/shop/onepagecheckout/onepage/updateData/

Response
https://gist.github.com/Alex-302/21ba2cbe8adce51f6180

http://www.aipa.ru/modules/blocklayered/blocklayered-ajax.php?id_category_layered=192&orderby=date_add&orderway=desc&n=20&p=1&_=1455181164940

Response
https://gist.github.com/Alex-302/76a25da989cafbb7c11c
content-type: text/html; charset=utf-8

@Alex-302 Alex-302 added the bug label Feb 10, 2016
@ameshkov ameshkov added this to the 6.0 R2 milestone Feb 12, 2016
@Alex-302
Copy link
Member Author

@ameshkov ameshkov changed the title Unnecessary Adguard injections Fix HTML content detection Mar 4, 2016
@ameshkov
Copy link
Member

ameshkov commented Mar 4, 2016

Fixed, review ID:
CR-AG-737

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants