Skip to content

[Auth] Add fastify extension, onToken callback, & secure option #1801

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
Merged
merged 14 commits into from
Oct 19, 2023
Merged

[Auth] Add fastify extension, onToken callback, & secure option #1801

merged 14 commits into from
Oct 19, 2023

Conversation

adam-maj
Copy link
Contributor

@adam-maj adam-maj commented Oct 19, 2023 •
edited
Loading

What's Changed

  • Added a @thirdweb-dev/auth/fastify entrypoint with a fastify plugin
  • All callbacks now have req: Request instead of req?: Request
  • Added an onToken callback

@adam-maj adam-maj requested a review from a team October 19, 2023 01:06
@changeset-bot
Copy link

changeset-bot bot commented Oct 19, 2023 •
edited
Loading

🦋 Changeset detected

Latest commit: 36f075a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 7 packages
Name Type
@thirdweb-dev/auth Minor
thirdweb Patch
@thirdweb-dev/react-core Patch
@thirdweb-dev/react Patch
@thirdweb-dev/unity-js-bridge Patch
@thirdweb-dev/react-native Patch
@thirdweb-dev/react-native-compat Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@adam-maj
Copy link
Contributor Author

/release-pr

@codecov
Copy link

codecov bot commented Oct 19, 2023 •
edited
Loading

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Files Coverage Δ
.../contracts/prebuilt-implementations/marketplace.ts 73.75% <100.00%> (ø)
...src/evm/contracts/prebuilt-implementations/pack.ts 67.66% <100.00%> (-0.39%) ⬇️
...rc/evm/contracts/prebuilt-implementations/split.ts 86.17% <100.00%> (-0.70%) ⬇️
...vm/contracts/prebuilt-implementations/multiwrap.ts 84.84% <50.00%> (-0.31%) ⬇️
...src/evm/contracts/prebuilt-implementations/vote.ts 71.42% <85.71%> (+0.14%) ⬆️

📢 Thoughts on this report? Let us know!.

@adam-maj
Copy link
Contributor Author

/release-pr

@adam-maj
Copy link
Contributor Author

/release-pr

@adam-maj
Copy link
Contributor Author

/release-pr

@adam-maj adam-maj changed the title Add fastify extension, onToken callback, & secure option [Auth] Add fastify extension, onToken callback, & secure option Oct 19, 2023
@socket-security
Copy link

socket-security bot commented Oct 19, 2023 •
edited
Loading

No top level dependency changes detected. Learn more about Socket for GitHub ↗︎

@adam-maj
Copy link
Contributor Author

/release-pr

@adam-maj
Copy link
Contributor Author

/release-pr

@adam-maj
Copy link
Contributor Author

/release-pr

@adam-maj adam-maj added this pull request to the merge queue Oct 19, 2023
Merged via the queue into main with commit 99d7ec0 Oct 19, 2023
@adam-maj adam-maj deleted the am/auth branch October 19, 2023 23:38
@github-actions github-actions bot mentioned this pull request Oct 19, 2023
Merged
jnsdls pushed a commit that referenced this pull request Jun 19, 2024
@assimovt
feat: support ach + payment verification (#1801)
48e9c9a 
fixes FTD-2200
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joaquim-verges joaquim-verges Awaiting requested review from joaquim-verges joaquim-verges is a code owner

@jnsdls jnsdls Awaiting requested review from jnsdls jnsdls is a code owner

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

Successfully merging this pull request may close these issues.
1 participant
@adam-maj