Skip to content

fix: missing headers for feeds mdb-3237 to mdb-3293 and mdb-2939#1756

Merged
cka-y merged 2 commits into
mainfrom
hotfix/missing_header
Jul 7, 2026
Merged

fix: missing headers for feeds mdb-3237 to mdb-3293 and mdb-2939#1756
cka-y merged 2 commits into
mainfrom
hotfix/missing_header

Conversation

@cka-y

@cka-y cka-y commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Summary:

This pull request introduces a hotfix to address issues with downloading certain GTFS feeds whose producers block the default HTTP headers. The main change is the addition of a custom http_headers override for a specific range of feeds.

Database migration and configuration:

  • Added a new migration, hotfix_missing_header.sql, to insert a custom User-Agent and Referer header override for feeds with stable IDs from mdb-3237 to mdb-3293 and mdb-2939, ensuring successful downloads from producers that block the default headers. (liquibase/changes/hotfix_missing_header.sql)
  • Updated the main Liquibase changelog (changelog.xml) to include the new hotfix migration, ensuring it is applied in the correct order. (liquibase/changelog.xml)
    Please make sure these boxes are checked before submitting your pull request - thanks!
  • Run the unit tests with ./scripts/api-tests.sh 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)

@cka-y cka-y marked this pull request as ready for review July 7, 2026 19:44
@cka-y cka-y requested a review from ianktc July 7, 2026 20:02
@cka-y cka-y merged commit fb05c49 into main Jul 7, 2026
21 of 22 checks passed
@cka-y cka-y deleted the hotfix/missing_header branch July 7, 2026 20:21
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