FiltersHeroes / PolishCookieConsent

Polish Cookie Consent is an extension, which automatically accepts privacy policy/GDPR on websites.
GNU General Public License v3.0
27 stars 0 forks source link

Baza filtrów cofa czasem się do wbudowanej w ostatnią wersje #72

Closed krystian3w closed 3 years ago

krystian3w commented 3 years ago

Sam mam to co jakiś czas, najczęściej zbiega się z aktualizacją Firefox.

Komentarz w Chrome Web Store może wskazuje, że "użytkowniczka" też to miała np. bo śledzi co zostało dodane po czerwcu 2020.

hawkeye116477 commented 3 years ago

Ta, najwyraźniej problem jest tu => https://github.com/PolishFiltersTeam/PolishCookieConsent/blob/master/src/update.js#L18.

Trzeba będzie doprecyzować => https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/runtime/OnInstalledReason, czyli tylko install i update.

krystian3w commented 3 years ago

A takie coś pozwoli na prace w ramkach?

  "content_scripts": [
    {
      "matches": [
        "http://*/*",
        "https://*/*"
      ],
      "js": [
        "content.js"
      ],
+++   "all_frames": true,
      "run_at": "document_start"
    }
  ],

Niby raz na rok się zdarza pop-up używający iframe, w takich żadne klikanie nie działało i trzeba było polegać na czytelności cookies/localStorage.

Chyba, że bezpieczniej dodawać nieczytelne dla ludzi wpisy w cookies/localStorage.

Po potem zostaje, że ogranicza nas to:

https://github.com/PolishFiltersTeam/PolishCookieConsent/blob/e42822e2de4c892800eadec84921cab1ccf82972/src/content.js#L1-L12

tam trudniej mi ustalić co dyskwalifikuje przechwycenie url ramki w celu weryfikacji czy filtr ma się wykonać.

hawkeye116477 commented 3 years ago

https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/content_scripts

Note: This also applies to any tracker or ad that uses iframes, which means that enabling this could make your content script get called dozens of times on some pages.

krystian3w commented 3 years ago

A naprawiamy bug z z wklejaniem np. || do listy wyjątków lub konwersji pustych linii w ||| np.:

abc.pl

def.pl

działa jak abc.pl|||def.pl

Próbowałem konwertować || do | ale jakby o etap za późno lub redukcja ||||| robiła się raz do |||. Niby obecność działa jak użycie @@*$ehide,shide (czyli filtry nie próbują klikać/piec ciastek/robić przekierowań).