Skip to content

test(auth): specs for the jwt + github strategies - #673

Open
yggdrasilworld wants to merge 1 commit into
MergeFi:mainfrom
yggdrasilworld:spec/auth-strategies-593-594
Open

yggdrasilworld wants to merge 1 commit into
MergeFi:mainfrom
yggdrasilworld:spec/auth-strategies-593-594

Conversation

@yggdrasilworld

Copy link
Copy Markdown

What

  • jwt.strategy — cookieExtractor (cookie-parsing + the decodeURIComponent guard — a malformed token returns null) + validate() rejecting tokens for deleted users.
  • github.strategy — the profile → upsert-input mapping (email/avatar, nulls when absent).

exported for the spec. Not runnable locally (deps) — verified semantically.

Resolves #593, #594

- jwt.strategy: the cookieExtractor (cookie-parsing + the decodeURIComponent
  guard — a malformed token returns null not a throw) and validate()
  rejecting tokens for deleted users.
- github.strategy: the profile → UpsertFromGithubInput mapping (email/avatar
  from the first entries, nulls when absent).

cookieExtractor exported for the spec. Semantics verified against the real
methods; the project jest isn't runnable in-env (deps).

Resolves MergeFi#593, MergeFi#594
@vercel

vercel Bot commented Oct 14, 2026

Copy link
Copy Markdown

@yggdrasilworld is attempting to deploy a commit to the chonilius' projects Team on Vercel.

A member of the Team first needs to authorize it.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

auth/strategies/jwt.strategy.ts has no spec file

1 participant