8000 update(api-key): Disable hashing API Keys by ping-maxwell · Pull Request #2373 · better-auth/better-auth · GitHub
[go: up one dir, main page]
Skip to content

update(api-key): Disable hashing API Keys #2373

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
< 8000 p class="mt-4 color-fg-muted text-center">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 6 commits into from
May 9, 2025
Merged

update(api-key): Disable hashing API Keys #2373

merged 6 commits into from
May 9, 2025

Conversation

ping-maxwell
Copy link
Collaborator
@ping-maxwell ping-maxwell commented Apr 20, 2025 •
edited
Loading

Allows the user to disable hashing of api keys.

  • docs
  • tests

For further context:
https://discord.com/channels/1288403910284935179/1288403910284935182/1363454834740559952

@ping-maxwell
update(api-key): Custom hashing function for api-keys
63b8864 
Useful if the user wants to use a custom hashing algorithm, or if they want to disable hashing, they can just return the key itself.
@vercel Vercel
Copy link
vercel bot commented Apr 20, 2025

@ping-maxwell is attempting to deploy a commit to the better-auth Team on Vercel.

A member of the Team first needs to authorize it.

@pkg-pr-new pkg.pr.new
Copy link
pkg-pr-new bot commented Apr 20, 2025 •
edited
Loading

Open in StackBlitz

npm i https://pkg.pr.new/better-auth/better-auth@2373

npm i https://pkg.pr.new/better-auth/better-auth/@better-auth/cli@2373

npm i https://pkg.pr.new/better-auth/better-auth/@better-auth/expo@2373

npm i https://pkg.pr.new/better-auth/better-auth/@better-auth/stripe@2373

commit: c977813

mrge-io[bot]
mrge-io bot reviewed Apr 20, 2025
View reviewed changes
Copy link
@mrge-io mrge-io bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mrge found 5 issues across 6 files. View them in mrge.io

@ping-maxwell ping-maxwell added the enhancement New feature or request label Apr 23, 2025
@ping-maxwell ping-maxwell changed the title update(api-key): Custom hashing function for api-keys update(api-key): Disable hashing API Keys Apr 25, 2025
@vercel Vercel
Copy link
vercel bot commented Apr 25, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
better-auth-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 25, 2025 7:15pm

@ping-maxwell ping-maxwell requested a review from Bekacru April 26, 2025 04:11
@Bekacru Bekacru merged commit cb1cbce into better-auth:main May 9, 2025
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mrge-io mrge-io[bot] mrge-io[bot] left review comments

@Bekacru Bekacru Awaiting requested review from Bekacru

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ping-maxwell @Bekacru
0