search

dyne / slangroom

Enhance zencode smart contracts with your slang
https://dyne.org/slangroom/
GNU Affero General Public License v3.0
5 stars 1 forks source link

Oauth "get claims from token" is no more needed #154

Closed matteo-cristino closed 5 months ago

matteo-cristino commented 5 months ago

The statement

https://github.com/dyne/slangroom/blob/dabdcd3d7fd892e579de8cdedd76c7dadd3c3db8/pkg/oauth/src/plugin.ts#L356-L380

is no more needed since the claims should not be saved in the authorization details anymore, however this statement is still usefull in order to get the authorization details. Thus I propose to modify it to

Given I send token 'token' and send server_data 'server_data' and get authorization details from token
puria commented 5 months ago

Great, I've assiged this to @RebeccaSelvaggini