Resolve some problems with fiware oauth2 - Githubissues

apinf / apinf-fiware

FIWARE IdM OAuth2 handler for Meteor.

European Union Public License 1.1

1 stars 3 forks source link

Resolve some problems with fiware oauth2 #9

Closed vellames2 closed 7 years ago

vellames2 commented 7 years ago

Closes #8

[x] Error trying to get the authorization code
[x] Error trying to get the access token
[x] Error trying to get the basic informations of user (getAccounts)

vellames2 commented 7 years ago

@mauriciovieira or @midyan please review the code

midyan commented 7 years ago

@Vellames I have committed the changes I did to the fiware_server.js to this branch. This is the new dynamic header: https://github.com/apinf/apinf-fiware/blob/2dbea1d68bd7e20e6b634ee1ee7a3cbcbcd9cad4/fiware_server.js#L110

https://github.com/apinf/apinf-fiware/blob/2dbea1d68bd7e20e6b634ee1ee7a3cbcbcd9cad4/fiware_server.js#L126-L129

The error I mentioned on #11 throws an exception here: https://github.com/apinf/apinf-fiware/blob/2dbea1d68bd7e20e6b634ee1ee7a3cbcbcd9cad4/fiware_server.js#L132-L136

That is why the getAccounts method is not being called. The accessToken error is blocking the rest of the code:

https://github.com/apinf/apinf-fiware/blob/2dbea1d68bd7e20e6b634ee1ee7a3cbcbcd9cad4/fiware_server.js#L44-L56