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

fix: swagger doc url behind reverse proxy #55

Merged
merged 1 commit into from
Aug 23, 2021
Merged

fix: swagger doc url behind reverse proxy #55

merged 1 commit into from
Aug 23, 2021

Conversation

luwol03
Copy link
Member

@luwol03 luwol03 commented Aug 20, 2021 •
edited
Loading

Status Type Env Vars Change
✅ Ready Bug No

Description

Fix bug if server is behind a reverse proxy the swagger current server url was wrong.

Motivation and Context

Screenshots / GIFs (if appropriate):

Checklist

  • I have read the CONTRIBUTING document.
  • I have considered the accessibility of my changes (i.e. did I add proper content descriptions to images, or run my changes with talkback enabled?)
  • I have documented my code if needed

Resolves

@luwol03 luwol03 added the kind/bug Something isn't working label Aug 20, 2021
@luwol03 luwol03 added this to the v1.1.0 milestone Aug 20, 2021
@luwol03 luwol03 requested a review from noctera August 20, 2021 09:21
@luwol03 luwol03 merged commit 8001ed2 into experimental Aug 23, 2021
@luwol03 luwol03 deleted the fix/swagger-doc-url branch August 26, 2021 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@noctera noctera noctera approved these changes

Assignees
No one assigned
Labels
kind/bug Something isn't working
Projects
Todo
Status: 🚀 Ready
Milestone
v1.1.0
Development

Successfully merging this pull request may close these issues.
2 participants
@luwol03 @noctera