Update content.js - Githubissues

mlgualtieri / CSS-Exfil-Protection

Official repository for the CSS Exfil Protection browser extensions.

MIT License

155 stars 11 forks source link

Update content.js #2

Closed earthlng closed 6 years ago

earthlng commented 6 years ago

missing // in detection, see https://no-csp-css-keylogger.badsite.io/
console.log can temporarily hang the browser if there are 1000s of rules, see https://no-csp-css-keylogger.badsite.io/
only update the badge once

With these changes the addon successfully blocks the keylogger PoC @ https://no-csp-css-keylogger.badsite.io/

mlgualtieri commented 6 years ago

Thank you! I actually just discovered that site and those same issues today. Will review your pull request asap and get an update out. Unsure why I didn't receive a notification I had a pull request. I really appreciate your help!

earthlng commented 6 years ago

You're welcome. Thank you for creating the addon in the 1st place. No idea why you didn't get a notification either, I should have included your nickname in the commit message I guess. I don't do pull requests very often but I'll keep that in mind for the next time.