conwetlab / ckanext-oauth2

OAuth2 support for CKAN
GNU Affero General Public License v3.0
25 stars 56 forks source link

Using the Authorization header on the get_token method is only required when using a legacy version of KeyRock #23

Closed aarranz closed 6 years ago

aarranz commented 6 years ago

Current implementation provides an Authorization header with the client id and client secret using base64 encoding.

We have tested the following services with the following results:

This PR change the code so this header is only used when connecting to a legacy version of the IdM. Should fix #22.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.007%) to 99.559% when pulling 9560fd68f1f2495c92a20458dd14adf1f50f6fe6 on aarranz:fix/wso2-get-token into 3749d7d7349bffac52a64c6ee9ae766d811d5b1c on conwetlab:master.