-
Notifications
You must be signed in to change notification settings - Fork 192
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
Remove authentication from /openapi/v2 call #372
Conversation
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.
Thanks for tracking down this issue and sending in a fix!
lib/swagger-client.js
Outdated
url: this.http.requestOptions.baseUrl, | ||
ca: this.http.requestOptions.ca | ||
}) | ||
return noAuthHTTP.http({ method: 'GET', pathname }) |
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.
What if we added a noAuth
option to http
? One benefit of doing that is if we modify Request
or add a new version and a user constructs it with some custom configuration that we wouldn't know how to replicate when we construct it here.
Thoughts?
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.
That sounds good to me, would it then be accessible through this.http.noAuth.http(...)
or do you have a better idea? Or am I misunderstanding?
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.
@@ -33,14 +33,14 @@ const Request = require('./request') | |||
|
|||
class Root extends Component { | |||
_getSpec (pathname) { | |||
return this.http.http({ method: 'GET', pathname }) |
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.
I was imagining something like:
return this.http.http({ method: 'GET', pathname, noAuth: true })
A little swamped this week but will get to this over the weekend. |
@silasbw Addressed comments in latest PR |
Thanks again. Published with |
fixes #371