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

Verify token when SDK starts #30

Merged
merged 4 commits into from
Sep 10, 2020
Merged

Verify token when SDK starts #30

merged 4 commits into from
Sep 10, 2020

Conversation

orzechdev
Copy link
Contributor

@orzechdev orzechdev commented Sep 5, 2020 •
edited
Loading

It implements token verification. Might be tested with saleor/saleor-storefront#883. Please review first #29.

@orzechdev orzechdev mentioned this pull request Sep 5, 2020
Merged
5 tasks
@orzechdev orzechdev changed the base branch from feature/jwt-expiration to master September 7, 2020 08:17
@patrys
Copy link
Member

patrys commented Sep 7, 2020

Task linked: CU-6a5vxt Storefront: use refresh Token to re-generate expired JWT token

@orzechdev orzechdev marked this pull request as ready for review September 7, 2020 13:43
AlicjaSzu
AlicjaSzu approved these changes Sep 9, 2020
View reviewed changes
src/state/index.ts Outdated Show resolved Hide resolved
@dominik-zeglen dominik-zeglen merged commit d0bc068 into master Sep 10, 2020
@dominik-zeglen dominik-zeglen deleted the feature/jwt-verify branch September 10, 2020 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dominik-zeglen dominik-zeglen dominik-zeglen approved these changes

@AlicjaSzu AlicjaSzu AlicjaSzu approved these changes

@krzysztofwolski krzysztofwolski Awaiting requested review from krzysztofwolski

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@orzechdev @patrys @dominik-zeglen @AlicjaSzu