search

uafrica / oauth-server

OAuth Server
Other
51 stars 51 forks source link

Redirected /oauth to /oauth/authorize #47

Closed sukihub closed 7 years ago

sukihub commented 7 years ago

Having separate /oauth route seems redundant, since it only:

  1. validates OAuth params
  2. performs login redirect

and both things are also done in authorize action.

I'd remove it completely, but that might break existing applications :)

Depends on #46

sukihub commented 7 years ago

Since we've talked about making it backward incompatible, how do you feel about deleting oauth action? :)

dakota commented 7 years ago

@sukihub Kill it with fire :) It's not doing anything useful.