Skip to content

fix: prevent html validation reports from being indexed by search engines - #2041

Merged
Alessandro100 merged 1 commit into
masterfrom
add-noindex-meta-tag
Jun 23, 2025
Merged

fix: prevent html validation reports from being indexed by search engines#2041
Alessandro100 merged 1 commit into
masterfrom
add-noindex-meta-tag

Conversation

@Alessandro100

Copy link
Copy Markdown
Contributor

This change adds a meta tag to the HTML report template to prevent search engines from indexing or following links in the report.

Summary:

This change adds a meta tag to the HTML report template to prevent search engines from indexing or following links in the report.

Expected behavior:

Google or other search engines should not index validation reports

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Run the unit tests with gradle test to make sure you didn't break anything
  • Add or update any needed documentation to the repo
  • Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
  • Linked all relevant issues
  • Include screenshot(s) showing how this pull request works and fixes the issue(s)

This change adds a meta tag to the HTML report template to prevent search engines from indexing or following links in the report.
@Alessandro100 Alessandro100 self-assigned this Jun 23, 2025
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@davidgamez davidgamez left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions

Copy link
Copy Markdown
Contributor

📝 Acceptance Test Report

📋 Summary

✅ The rule acceptance has passed for commit d708c0f
Download the full acceptance test report here (report will disappear after 90 days).

📊 Notices Comparison

New Errors (0 out of 1913 datasets, ~0%) ✅

No changes were detected due to the code change.

Dropped Errors (0 out of 1913 datasets, ~0%) ✅

No changes were detected due to the code change.

New Warnings (0 out of 1913 datasets, ~0%) ✅

No changes were detected due to the code change.

Dropped Warnings (0 out of 1913 datasets, ~0%) ✅

No changes were detected due to the code change.

🛡️ Corruption Check

39 out of 1952 sources (~2 %) are corrupted.
Dataset Ref Report Exists Ref Report Readable Latest Report Exists Latest Report Readable
au-australian-capital-territory-transport-canberra-gtfs-2675
ca-saskatchewan-moose-jaw-transit-gtfs-2602
de-baden-wuttemberg-nvbw-gtfs-2393.json
de-bayern-munchner-verkehrs--und-tarifverbund-gmbh-mvv-gtfs-2252
de-sachsen-mitteldeutscher-verkehrsverbund-gmbh-mdv-gtfs-2360
fr-eurostar-gtfs-2431.json
fr-grand-est-troyes-champagne-metropole-tcat-gtfs-2627
fr-grand-est-troyes-champagne-metropole-tcat-gtfs-2628
ie-dublin-bus-eireann-gtfs-2636
il-ministry-of-transport-and-road-safety-gtfs-2519
jp-aomori-aomori-city-bus-gtfs-2607
ro-buzau-transbus-buzau-gtfs-2106
ro-dambovita-servicii-publice-municipale-targoviste-gtfs-2107
ro-prahova-transport-calatori-express-ploiesti-gtfs-2108
us-california-alameda-contra-costa-transit-district-ac-transit-gtfs-2455
us-california-county-connection-gtfs-2421
us-california-regional-transportation-commission-of-southern-nevada-rtc-gtfs-110
us-california-santa-maria-area-transit-gtfs-26
us-california-south-county-transit-link-gtfs-2203
us-california-taft-maricopa-area-transit-gtfs-821
us-california-tri-delta-transit-gtfs-1974
us-florida-gainesville-regional-transit-system-gtfs-2412
us-hawaii-hawaii-mass-transit-agency-hele-on-bus-gtfs-2608
us-illinois-danville-mass-transit-gtfs-2363
us-kansas-salina-gtfs-1867
us-massachusetts-pioneer-valley-transit-authority-pvta-gtfs-2416
us-utah-cache-valley-gtfs-1906
us-virginia-arlington-transit-gtfs-485
us-virginia-fairfax-cue-bus-cue-gtfs-2609
us-virginia-fredericksburg-regional-transit-gtfs-2430
us-washington-coast-transportation-gtfs-2162
us-washington-columbia-county-public-transportation-gtfs-2168
us-washington-community-in-motion-gtfs-2163
us-washington-eastside-friends-of-seniors-gtfs-2166
us-washington-hopelink-transportation-gtfs-2167
us-washington-intercity-transit-gtfs-2289
us-washington-paratransit-services-gtfs-2176
us-washington-puget-sound-educational-service-district-gtfs-2177
us-washington-sound-generations-hyde-shuttle-gtfs-2183

⏱️ Performance Assessment

📈 Validation Time

Assess the performance in terms of seconds taken for the validation process.

Time Metric Dataset ID Reference (s) Latest (s) Difference (s)
Average -- 3.77 3.87 ⬆️+0.10
Median -- 1.43 1.51 ⬆️+0.08
Standard Deviation -- 10.53 10.56 ⬆️+0.02
Minimum in References Reports us-california-catalina-express-gtfs-299 0.50 0.74 ⬆️+0.24
Maximum in Reference Reports gb-unknown-uk-aggregate-feed-gtfs-2014 267.32 266.89 ⬇️-0.43
Minimum in Latest Reports ph-unknown-hm-transport-inc-and-robinsons-malls-gtfs-1105 0.50 0.57 ⬆️+0.07
Maximum in Latest Reports gb-unknown-uk-aggregate-feed-gtfs-2014 267.32 266.89 ⬇️-0.43
📜 Memory Consumption
Metric Dataset ID Reference (s) Latest (s) Difference (s)
Average -- 478.05 MiB 471.77 MiB ⬇️-6.29 MiB
Median -- 334.25 MiB 334.68 MiB ⬆️+434.77 KiB
Standard Deviation -- 804.02 MiB 776.95 MiB ⬇️-27.07 MiB
Minimum in References Reports mexico-jalisco-direccion-general-de-transporte-publico-de-puerto-vallarta-gtfs-2034 38.35 MiB 391.93 MiB ⬆️+353.58 MiB
Maximum in Reference Reports ch-unknown-swiss-federal-railways-sbb-gtfs-2144 8.43 GiB 8.71 GiB ⬆️+289.22 MiB
Minimum in Latest Reports us-virginia-petersburg-area-transit-flex-gtfs-2441 51.32 MiB 38.58 MiB ⬇️-12.74 MiB
Maximum in Latest Reports ch-unknown-swiss-federal-railways-sbb-gtfs-2144 8.43 GiB 8.71 GiB ⬆️+289.22 MiB

@Alessandro100
Alessandro100 merged commit 435558c into master Jun 23, 2025
@welcome

welcome Bot commented Jun 23, 2025

Copy link
Copy Markdown

🥳 Congrats on getting your first pull request merged!

@Alessandro100
Alessandro100 deleted the add-noindex-meta-tag branch June 23, 2025 13:45
sylvansson pushed a commit to sylvansson/gtfs-validator that referenced this pull request Jul 8, 2025
This change adds a meta tag to the HTML report template to prevent search engines from indexing or following links in the report.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants