Skip to content

Make sure the authenticator has accessToken, refreshToken, user data #76

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
Open
check-cafe opened this issue Nov 9, 2024 · 0 comments
Open

Make sure the authenticator has accessToken, refreshToken, user data #76

check-cafe opened this issue Nov 9, 2024 · 0 comments
Assignees
@mhaidarhanif @zckyachmd @difasulthon @Endabelyu @IQchaidir

Comments

@check-cafe
Copy link
Contributor

This will fix race conditions in some places.

// We should only rely on this authenticator
const authenticated = await authenticator.isAuthenticated(request);

// TODO: This might cause race conditions
const { accessToken, headers } = await getAccessToken(request);
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mhaidarhanif mhaidarhanif

@zckyachmd zckyachmd

@difasulthon difasulthon

@Endabelyu Endabelyu

@IQchaidir IQchaidir

Labels
None yet
Projects
CheckCafe
Status: Todo: API
Milestone
No milestone
Development

No branches or pull requests
6 participants
@mhaidarhanif @zckyachmd @difasulthon @Endabelyu @IQchaidir @check-cafe