-
-
Notifications
You must be signed in to change notification settings - Fork 229
fix: Allow tokenUrl to be relative #618
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.
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
Conversation
Looks like the change before: openapi-generators@15c2fd5#diff-1a1a2865f336a376229eaffe0f56caac01c7a3a77602e07fe4a38fd550c6c318 Has been reverted: openapi-generators@0101a03#diff-6676609c9c37705f6953ffcc9eb6d33164df50f241d4844c5665cf5ff4305995R16 As mentioned, relative URLs are supported in OpenAPI 3.0+: swagger-api/swagger-ui#5243
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks for the fix. I'll approve CI to make sure nothing errors there.
Codecov Report
@@ Coverage Diff @@
## main #618 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 49 49
Lines 1713 1713
=========================================
Hits 1713 1713
Continue to review full report at Codecov.
|
Just ran into this issue with fastapi-users usage . Will this be added to an official release soon? Thanks. @dbanty |
All set 👍 |
Looks like the change before:
15c2fd5#diff-1a1a2865f336a376229eaffe0f56caac01c7a3a77602e07fe4a38fd550c6c318
Has been reverted:
0101a03#diff-6676609c9c37705f6953ffcc9eb6d33164df50f241d4844c5665cf5ff4305995R16
As mentioned, relative URLs are supported in OpenAPI 3.0+:
swagger-api/swagger-ui#5243