-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
Erroneous errors in latest Swagger UI version #9208
Comments
There might be an issue with the latest update of the swagger-client and related code in /core/index.js d89726a |
Hi everybody, The regression will most likely be in swagger-client which was recently heavily refactored. Specifically we've changed how URLs are resolved. Instead of legacy and deprecated url we now use https://url.spec.whatwg.org/. Let me investigate what went wrong and why. |
I can reproduce by passing SwaggerUI({
spec: {...},
}) |
Addressed in #9214. Explanation: if we pass Before we were running resolution on relative URI References, which was just wrong. |
Q&A (please complete the following information)
Content & configuration
Example Swagger/OpenAPI definition (redacted):
Swagger-UI configuration options:
Describe the bug you're encountering
Erroneous error messages are showing up at the the top of the documenation page. This happens in
5.7.0
but not in5.6.2
.To reproduce...
Steps to reproduce the behavior:
Expected behavior
No errors should appear
Screenshots
Additional context or thoughts
I'm having trouble tracing where the error is happening based on the browser console but based on the stack trace it seems to be originating from the @swagger-api/apidom-reference in the
url.mjs
moduleThe text was updated successfully, but these errors were encountered: