OAuth1 icon indicating copy to clipboard operation
OAuth1 copied to clipboard

Published 20 hours ago • verified publisher icon WP-API

Run `authenticate` in `get_authentication_errors`

Open adamsilverstein opened this issue 9 years ago • 2 comments

Fixes an issue where logged in users could not authenticate requests with their OAuth credentials.

  • Removes hooking of authenticate on determin_current_user (which doesn't run or runst too late when the user presents login cookies)
  • Instead, authenticate directly in rest_authentication_errors callback

adamsilverstein avatar Mar 02 '16 20:03 adamsilverstein

any option on this @rmccue? it fixes the logged in user issue.

adamsilverstein avatar Mar 10 '16 00:03 adamsilverstein

I think this might have got fixed in here : https://github.com/WP-API/OAuth1/commit/662c9f840760b1c5c7092acfb94bd3ae2703256a

All OAuth tests seem to be working for me now.

My cobbled-together test script is here : https://gist.github.com/kosso/71c957e30a40116e5f98

kosso avatar Mar 10 '16 00:03 kosso