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

you can use basic auth AND be logged into the browser #18

Closed
wants to merge 3 commits into from

Conversation

mnelson4
Copy link

@mnelson4 mnelson4 commented Apr 8, 2015

addresses #12 in WP-API/Basic-Auth. Just required that the basic-auth take a stab at finding the current user, and if it does find the current user, then it we shouldn't have the cookie method try finding the current user separately (where it would overwrite what we've found).
I've been using this while logged into my browser and using basic auth.

@mnelson4
Copy link
Author

fyi someone else confirmed that this branch works: eventespresso/eea-rest-api#37 (comment)
so it would be nice to get this merged

@ganeshcodes
Copy link

I have tested https://github.com/eventespresso/Basic-Auth this. Basic authentication works after installing this plugin while the original basic auth plugin didn't seem to work for me. I guess it needs to be merged.

Mike Nelson added 2 commits February 24, 2016 16:28
@mnelson4
Copy link
Author

I'm replacing this pull request with #30

@mnelson4 mnelson4 closed this Feb 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants