chore: Bloquer plus d'extension de fichiers et de domaines de tracking by yann120 · Pull Request #598 · betagouv/acces-cible

yann120 · 2026-06-19T15:27:52Z

Cette PR étend la blocklist utilisée par le navigateur d’audit pour éviter de charger des ressources inutiles à l’analyse.

Changements principaux :

partage des listes d’extensions entre Browser et Page
blocage des documents téléchargeables courants : PDF, Office, OpenDocument, texte, CSV/TSV, iWork
blocage de ressources statiques supplémentaires : polices, images, audio, vidéo, flux, calendriers
ajout de domaines de tracking courants rencontrés pendant le scraping
ajout d’un pattern Sentry générique pour couvrir sentry.io, sentry.incubateur.net et autres variantes similaires

Josyann · 2026-06-22T07:44:55Z

Super ! Mais on avait discuté du fait de bloquer d'autres extensions aussi (listées dans page.rb) si possible/besoin. Tu as pris le temps de regarder ?

Josyann · 2026-06-22T08:07:23Z

J'y pense mais y'a sentry.incubateur.net aussi

Josyann · 2026-06-24T16:04:05Z

+  DOCUMENT_EXTENSIONS = [
+    # PDF and archives
+    %w[.pdf .zip],
+    # OpenDocument and Microsoft Office
+    %w[.odt .ods .odp .odg .doc .docx .xls .xlsx .ppt .pptx],
+    # Text, data, and Apple iWork
+    %w[.rtf .txt .csv .tsv .pages .numbers],
+  ].flatten.freeze
+
+  FILE_EXTENSIONS = [
+    # Fonts
+    %w[.woff .woff2 .ttf .otf .eot],
+    # Feeds, structured data, calendars, icons, and cursors
+    %w[.xml .rss .atom .ics .ical .ico .cur],
+    # Images
+    %w[.jpg .jpeg .png .gif .bmp .svg .webp .avif .tif .tiff .apng .heic .heif],
+    # Audio and video
+    %w[.mp3 .mp4 .avi .mov .mkv .webm .ogg .wav .aac .flac .m4a .opus],
+    %w[.ogv .m4v .mpg .mpeg],
+  ].flatten.freeze
+
+  BLOCKED_FILE_EXTENSIONS = (FILE_EXTENSIONS + DOCUMENT_EXTENSIONS).freeze



Suggested change

DOCUMENT_EXTENSIONS = [

# PDF and archives

%w[.pdf .zip],

# OpenDocument and Microsoft Office

%w[.odt .ods .odp .odg .doc .docx .xls .xlsx .ppt .pptx],

# Text, data, and Apple iWork

%w[.rtf .txt .csv .tsv .pages .numbers],

].flatten.freeze

FILE_EXTENSIONS = [

# Fonts

%w[.woff .woff2 .ttf .otf .eot],

# Feeds, structured data, calendars, icons, and cursors

%w[.xml .rss .atom .ics .ical .ico .cur],

# Images

%w[.jpg .jpeg .png .gif .bmp .svg .webp .avif .tif .tiff .apng .heic .heif],

# Audio and video

%w[.mp3 .mp4 .avi .mov .mkv .webm .ogg .wav .aac .flac .m4a .opus],

%w[.ogv .m4v .mpg .mpeg],

].flatten.freeze

BLOCKED_FILE_EXTENSIONS = (FILE_EXTENSIONS + DOCUMENT_EXTENSIONS).freeze

DOCUMENT_EXTENSIONS = [

# PDF and archives

'.pdf', '.zip',

# OpenDocument and Microsoft Office

'.odt', '.ods', '.odp', '.odg', '.doc', '.docx', '.xls', '.xlsx', '.ppt', '.pptx',

# Text, data, and Apple iWork

'.rtf', '.txt', '.csv', '.tsv', '.pages', '.numbers',

].freeze

FILE_EXTENSIONS = [

# Fonts

'.woff', '.woff2', '.ttf', '.otf', '.eot',

# Feeds, structured data, calendars, icons, and cursors

'.xml', '.rss', '.atom', '.ics', '.ical', '.ico', '.cur',

# Images

'.jpg', '.jpeg', '.png', '.gif', '.bmp', '.svg', '.webp', '.avif', '.tif', '.tiff', '.apng', '.heic', '.heif',

# Audio and video

'.mp3', '.mp4', '.avi', '.mov', '.mkv', '.webm', '.ogg', '.wav', '.aac', '.flac', '.m4a', '.opus',

'.ogv', '.m4v', '.mpg', '.mpeg',

].freeze

BLOCKED_FILE_EXTENSIONS = (FILE_EXTENSIONS + DOCUMENT_EXTENSIONS).freeze

bonne idée! 👍🏻

Josyann · 2026-06-24T16:22:58Z

-      next if uri.path && File.extname(uri.path).match?(FILES_EXTENSIONS)
-      next if skip_files && uri.path && File.extname(uri.path).match?(DOCUMENT_EXTENSIONS)
+      next if uri.path && Browser::FILE_EXTENSIONS.include?(File.extname(uri.path).downcase)
+      next if skip_files && uri.path && Browser::DOCUMENT_EXTENSIONS.include?(File.extname(uri.path).downcase)


Suggested change

next if skip_files && uri.path && Browser::DOCUMENT_EXTENSIONS.include?(File.extname(uri.path).downcase)

next if skipped_file_link?(uri, skip_files:)

[...]

private

def skipped_file_link?(uri, skip_files:)

return false unless uri.path

path_extension = File.extname(uri.path).downcase

Browser::FILE_EXTENSIONS.include?(path_extension) ||

skip_files && Browser::DOCUMENT_EXTENSIONS.include?(path_extension)

end

Ça rajoute des lignes mais je trouve ça plus lisible. Comme tu veux !

oui très bien, je suis pour la lisibilité 👍🏻

Josyann · 2026-06-30T09:09:24Z

Voir pour les <embed></embed>
http://heberville.fr/

Co-authored-by: Josyann <Josyann@users.noreply.github.com>

yann120 · 2026-06-30T13:21:22Z

Voir pour les <embed></embed> http://heberville.fr/

@Josyann j'ai testé avec seulement embed pour voir si ca a un impact, et ça en a pas du tout sur le navigateur car ces INVISIBLE_ELEMENTS servent a retirer des élements dans la fonction dom qui nous retourne le HTML que l'on sauvegarde.

Je n'ai pas trouvé de moyen "simple" de supprimer une partie du html, comme les attributs embed avant le chargement de la page...

mais la bonne nouvelle c'est que bloquer les extensions PDF (cette fois c'est fait coté Ferrum) ne charge pas ces fichiers qui sont dans le embed comme sur http://heberville.fr/ et donc on affiche pas les iFrame injectés par Chrome pour afficher son extension de lecteur de PDF.
Le scraping est aussi beaucoup rapide sur cette page avec cette PR : 1 seconde vs 30 a 60 secondes (peut etre le timeout)

yann120 · 2026-06-30T15:56:36Z

Comparaison sur 250 audits : pas de différence... 9,8 minutes pour cette branche et sur main

Mais de grosse differences quand on lance un seul site.
Exemple : http://heberville.fr/

0.583 seconds pour la version avec les extensions de fichiers bloqués (ici l'ajout du .pdf fait effet)
121.38 seconds sur main

chore: block pdf files in order to increase browser performance

face50a

yann120 marked this pull request as ready for review June 22, 2026 07:32

yann120 requested a review from Josyann June 22, 2026 07:32

yann120 added 2 commits June 22, 2026 16:48

chore: add new file extentions and refactor

d21b180

chore: add new tracking domains to the blocked list

14d617d

yann120 changed the title ~~chore: block pdf files in order to increase browser performance~~ chore: Bloquer plus d'extension de fichiers et de domaines de tracking Jun 24, 2026

yann120 requested review from Josyann and removed request for Josyann June 24, 2026 14:30

yann120 self-assigned this Jun 24, 2026

yann120 added the chore Technical task label Jun 24, 2026

yann120 linked an issue Jun 24, 2026 that may be closed by this pull request

[CHORE] Bloquer plus d'extension de fichiers et de domaines de tracking #603

Closed

Josyann reviewed Jun 24, 2026

View reviewed changes

Josyann approved these changes Jun 24, 2026

View reviewed changes

simplify blocked file extension handling

25963ba

Co-authored-by: Josyann <Josyann@users.noreply.github.com>

yann120 merged commit 8bbca89 into main Jul 1, 2026
10 checks passed

yann120 deleted the chore/block-pdf branch July 1, 2026 08:21

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

chore: Bloquer plus d'extension de fichiers et de domaines de tracking#598

chore: Bloquer plus d'extension de fichiers et de domaines de tracking#598
yann120 merged 4 commits into
mainfrom
chore/block-pdf

yann120 commented Jun 19, 2026 •

edited

Loading

Uh oh!

Josyann commented Jun 22, 2026

Uh oh!

Josyann commented Jun 22, 2026

Uh oh!

Josyann Jun 24, 2026

Uh oh!

yann120 Jun 30, 2026

Uh oh!

Josyann Jun 24, 2026

Uh oh!

yann120 Jun 30, 2026

Uh oh!

Josyann commented Jun 30, 2026 •

edited

Loading

Uh oh!

yann120 commented Jun 30, 2026

Uh oh!

yann120 commented Jun 30, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

-      next if skip_files && uri.path && Browser::DOCUMENT_EXTENSIONS.include?(File.extname(uri.path).downcase)
+next if skipped_file_link?(uri, skip_files:)
+[...]
+private
+def skipped_file_link?(uri, skip_files:)
+  return false unless uri.path
+  path_extension = File.extname(uri.path).downcase
+  Browser::FILE_EXTENSIONS.include?(path_extension) ||
+    skip_files && Browser::DOCUMENT_EXTENSIONS.include?(path_extension)
+end

Uh oh!

Conversation

yann120 commented Jun 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Josyann commented Jun 22, 2026

Uh oh!

Josyann commented Jun 22, 2026

Uh oh!

Josyann Jun 24, 2026

Choose a reason for hiding this comment

Uh oh!

yann120 Jun 30, 2026

Choose a reason for hiding this comment

Uh oh!

Josyann Jun 24, 2026

Choose a reason for hiding this comment

Uh oh!

yann120 Jun 30, 2026

Choose a reason for hiding this comment

Uh oh!

Josyann commented Jun 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

yann120 commented Jun 30, 2026

Uh oh!

yann120 commented Jun 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

yann120 commented Jun 19, 2026 •

edited

Loading

Josyann commented Jun 30, 2026 •

edited

Loading

yann120 commented Jun 30, 2026 •

edited

Loading