Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Local server does not take into account the data from the authorization token (AMAZON_COGNITO_USER_POOLS auth mode) #139

Closed
max-konin opened this issue May 17, 2023 · 2 comments · Fixed by #143
Closed

Local server does not take into account the data from the authorization token (AMAZON_COGNITO_USER_POOLS auth mode) #139

max-konin opened this issue May 17, 2023 · 2 comments · Fixed by #143
Assignees
@max-konin
Labels
is: feature request New feature request package: server Server package
Milestone
1.0.0-rc.7

Comments

@max-konin
Copy link
Contributor

Hi
I see that some data such as username & sub is hardcoded. And claims is an empty object. And there is no way to influence it.
I think it would be logical to take this data from a real Cognito token. I sketched a simple solution here max-konin@739e27b

What do you think?
@maoosi
Copy link
Owner

maoosi commented May 18, 2023

@max-konin I think that's a great idea! Feel free to open a PR and I will include your contribution to the next release.

@maoosi maoosi added is: feature request New feature request package: server Server package labels May 18, 2023
@maoosi maoosi added this to the 1.0.0-rc.7 milestone May 18, 2023
@max-konin max-konin mentioned this issue May 22, 2023
Merged
@maoosi
Copy link
Owner

maoosi commented May 31, 2023

Will be released as part of 1.0.0-rc.7

@maoosi maoosi closed this as completed May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@max-konin max-konin

Labels
is: feature request New feature request package: server Server package
Projects
Prisma-AppSync Roadmap
Status: Released
Milestone
1.0.0-rc.7
Development

Successfully merging a pull request may close this issue.

Use auth data from Cognito JWT max-konin/prisma-appsync
2 participants
@max-konin @maoosi