infra: normalize swagger servers[] to the production URL in download_swagger.py#56
Open
bzajzon-laserfiche wants to merge 1 commit into
Open
infra: normalize swagger servers[] to the production URL in download_swagger.py#56bzajzon-laserfiche wants to merge 1 commit into
bzajzon-laserfiche wants to merge 1 commit into
Conversation
…swagger.py Regenerating against a local server baked localhost:11211 into the swagger servers[] and thus every generated client baseUrl default (the recurring localhost-baking trap), requiring a manual restore each regen. download_swagger.py now forces servers[] to the canonical production URL by default (--canonical-server-url; pass an empty string to keep the source). index.ts now always defaults baseUrl to production; a local host is used only when a client is constructed with it explicitly (as the integration tests do).
alexgomezlf
approved these changes
Jun 12, 2026
bzajzon-laserfiche
had a problem deploying
to
github-pages
GitHub Actions
Failure
bzajzon-laserfiche
requested a deployment
to
npm_publish
GitHub Actions
Waiting
bzajzon-laserfiche
requested a deployment
to
npm_publish
GitHub Actions
Waiting
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Regen-tooling fix for the recurring localhost-baking trap: regenerating against a local server bakes
http://localhost:11211into the swaggerservers[]and thus every generated client baseUrl default, requiring a manual restore on each regen.download_swagger.pynow forcesservers[]to the canonical production URL by default (--canonical-server-url; pass an empty string to keep the source URL). The generated client continues to default baseUrl to production; a local host is used only when a client is explicitly constructed with one (as the integration tests do).Split out of the 6.4.B/C/D feature work (#55) per the 'infra fix ships in its own PR' convention — tooling-only, no change to the published client surface.