Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

OpenAPI spec wrong URL #161

Open
Hazmi35 opened this issue Oct 17, 2024 · 1 comment · May be fixed by #162
Open

OpenAPI spec wrong URL #161

Hazmi35 opened this issue Oct 17, 2024 · 1 comment · May be fixed by #162

Comments

@Hazmi35
Copy link

Hazmi35 commented Oct 17, 2024 •
edited
Loading

I was using Traefik with these match rules.

Host(`domain.com`) && PathPrefix (`/api`)

This will route all domain.com with the prefix /api to the desired service which is our backend running Elysia with swagger. With #16 it was fixed to use the prefix from Elysia configuration, but on the latest version the Swagger option url is in the wrong url. (Does not respect prefix at all).
Hazmi35 added a commit to hzmifork/elysia-swagger that referenced this issue Oct 17, 2024
@Hazmi35
fix: use requestPath for /json
2958f44 
Fixes elysiajs#161.

This will also work when you don't specify any prefix to Elysia but uses Traefik Middlewares to strip /api prefix too.
@Hazmi35 Hazmi35 linked a pull request Oct 17, 2024 that will close this issue
Open
@Hazmi35
Copy link
Author

Hazmi35 commented Oct 20, 2024

While #162 fixes externally set prefixes from Traefik for example, it does not respect Elysia prefix as stated in
#162 (comment)

@Hazmi35 Hazmi35 mentioned this issue Oct 20, 2024
Open
Hazmi35 added a commit to hzmifork/elysia-swagger that referenced this issue Nov 21, 2024
@Hazmi35
fix: use requestPath for /json
5f29a45 
Fixes elysiajs#161.

This will also work when you don't specify any prefix to Elysia but uses Traefik Middlewares to strip /api prefix too.
Hazmi35 added a commit to hzmifork/elysia-swagger that referenced this issue Nov 29, 2024
@Hazmi35
fix: use requestPath for /json
95168c0 
Fixes elysiajs#161.

This will also work when you don't specify any prefix to Elysia but uses Traefik Middlewares to strip /api prefix too.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Respect externally set prefixes hzmifork/elysia-swagger
1 participant
@Hazmi35