-
Notifications
You must be signed in to change notification settings - Fork 23
Conversation
return harValue('cookies', { | ||
name: security.name, | ||
value: values[scheme], | ||
}); |
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.
:easy:
|
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.
Really not sure if I want to ever support reading from We should have a larger discussion about that some point. |
I think if we had Link support we'd never need to read from document.cookie right? |
Kind of, but a (the?) problem with the Link spec right now is that it has no formal support for auth flows so there isn't quite a spec-friendly way to grab data from a response and re-use it in auth. |
🧰 What's being changed?
This adds support for being able to handle cookie auth throughout our general Explorer authentication flows.
It functions identical to how
apiKey
works, just that ifin
is set tocookie
, the payload will be sent as a cookie.Resolves https://github.com/readmeio/api-explorer/issues/89
🧪 Testing
I've added an addition to our
auth-types
example and support can be seen there:For comparison this is what it looks like against
master
: