Skip to content

fix: prefer forwarded swagger base url#59

Merged
gabedalmolin merged 1 commit intomainfrom
fix/swagger-forwarded-base-url
Mar 11, 2026
Merged

fix: prefer forwarded swagger base url#59
gabedalmolin merged 1 commit intomainfrom
fix/swagger-forwarded-base-url

Conversation

@gabedalmolin
Copy link
Copy Markdown
Owner

Summary

  • prefer forwarded host/proto when deriving the public OpenAPI server URL
  • keep the local fallback for development and preserve the existing smoke guard
  • cover the forwarded-header behaviour in the Swagger contract tests

Validation

  • npm run typecheck
  • npm run lint
  • npm test
  • npm run build

Notes

  • follows up the previous Swagger fix so Railway public docs advertise https://... instead of http://...

@gabedalmolin gabedalmolin merged commit da78f7e into main Mar 11, 2026
5 checks passed
@gabedalmolin gabedalmolin deleted the fix/swagger-forwarded-base-url branch March 11, 2026 23:07
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.

1 participant