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.

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

Compatibility with credentialed requests #94

Merged
merged 1 commit into from
Oct 7, 2015

Conversation

Kingson-de
Copy link

In order that drakov supports credentialed requests there needs to be some tweaks to the control headers.
For example to use drakov together with the HAL browser this change is needed because the HAL browser can only send credentialed requests.

@yakovkhalinsky
Copy link
Contributor

@Kingson-de thank you for your PR.

Would be interesting to hear about how you are using Drakov.

Merging

yakovkhalinsky added a commit that referenced this pull request Oct 7, 2015
compatibility with credentialed requests
@yakovkhalinsky yakovkhalinsky merged commit 77275dd into Aconex:master Oct 7, 2015
@yakovkhalinsky yakovkhalinsky changed the title compatibility with credentialed requests Compatibility with credentialed requests Oct 7, 2015
@Kingson-de Kingson-de deleted the cors-with-credentials branch October 8, 2015 07:31
@Kingson-de
Copy link
Author

@yakovkhalinsky Thanks for this quick merge.
In my company we want to improve an existing API and are currently designing version 2.
So with drakov we have the great opportunity to design this new version with API blueprint and then test it with the HAL browser. Because one of the key features is to use hypermedia and also our app developers can already start working on version 2. Thanks for this great tool it helped us a lot and it was also part of many meetings here :)

@yakovkhalinsky
Copy link
Contributor

@Kingson-de great to hear some feedback!

I'll have this up on npm in the next 24 hours 👍

@Kingson-de
Copy link
Author

@yakovkhalinsky perfect!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants