Skip to content

Issue 89

Igor Alexandrov requested to merge igor-alexandrov/rodauth-oauth:issue-89 into master

Fixes #89 (closed).

Here is what has been done:

  • Before checking for supported auth method, now we ensure that oauth_application exists.
  • Also I removed Regex that has been used to extract token from Authorization header. The reason to do this is because you use Array.pack("m*") to encode header value, which actually uses Base64.encode64, which can add \n symbols to the encoded results. Your existing Regex does not work with \n in middle of the string and tests were falling back to authorization_required response. I removed Regex, maybe it does not look now as good as it was with Regex, but at least it works.
Edited by HoneyryderChuck

Merge request reports

Loading