-
Notifications
You must be signed in to change notification settings - Fork 84
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.
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
feat: Add getInitialSessionAuthContext to compute SessionAuth context #785
Changes from 4 commits
255d04f
032ed77
91c50d0
50b07ca
a651101
cbced55
7de5b21
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -18,6 +18,7 @@ import NormalisedURLDomain from "./normalisedURLDomain"; | |
import NormalisedURLPath from "./normalisedURLPath"; | ||
import { TypeFramework } from "./framework/types"; | ||
import { RecipeLevelUser } from "./recipe/accountlinking/types"; | ||
import { ClaimValidationError } from "./recipe/session/types"; | ||
import { BaseRequest } from "./framework"; | ||
|
||
export type AppInfo = { | ||
|
@@ -113,3 +114,13 @@ export type User = { | |
// the recipeUserId can be converted to string from the RecipeUserId object type. | ||
toJson: () => any; | ||
}; | ||
|
||
export type SSRSessionContextType = { | ||
isContextFromSSR: true; | ||
loading: false; | ||
Comment on lines
+118
to
+119
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. These should not be required since they are static values. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. But they are expected by |
||
doesSessionExist: boolean; | ||
accessTokenPayload: any; | ||
invalidClaims: ClaimValidationError[]; // TODO: Find if it's possible to compute invalidClaims on BE | ||
sasha240100 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
userId?: string; | ||
accessDeniedValidatorError?: ClaimValidationError; | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we need this function. And if we are going to keep it, why is it in the nextjs file?