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

addresses #12 in WP-API/Basic-Auth so you can use basic auth AND be l… #30

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mnelson4
Copy link

…ogged in the browser

replaces #18 (because the branch in that pull request now has content from another pull request)

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 mnelson4 closed this May 9, 2016
@mnelson4 mnelson4 deleted the cookie-auth-compatibility branch May 9, 2016 17:05
@mnelson4 mnelson4 restored the cookie-auth-compatibility branch May 12, 2016 18:36
@mnelson4 mnelson4 reopened this May 12, 2016
@mnelson4
Copy link
Author

oups didn't realize deleting the branch would close the pull request. duh

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.

1 participant