Skip to content

Conversation

coyotte508
Copy link
Contributor

@coyotte508 coyotte508 commented Apr 17, 2025

☕️ Reasoning

Add the Hugging Face provider. We already use NextAuth internally in https://endpoints.huggingface.co, and now some of our partners use NextAuth to integrate with Hugging Face.

So we want to make Hugging Face a built-in provider :)

See https://huggingface.co/spaces/huggingfacejs/inference-widgets/blob/main/packages/widgets/src/hooks.server.ts#L28 for a public existing use of NextAuth with Hugging Face.

🧢 Checklist

  • Documentation
  • Tests
  • Ready to be merged

🎫 Affected issues

📌 Resources

Copy link

vercel bot commented Apr 17, 2025

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

Name Status Preview Comments Updated (UTC)
auth-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 20, 2025 5:08am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
next-auth-docs ⬜️ Ignored (Inspect) Visit Preview Apr 20, 2025 5:08am

Copy link

vercel bot commented Apr 17, 2025

@coyotte508 is attempting to deploy a commit to the authjs Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added providers core Refers to `@auth/core` labels Apr 17, 2025
Copy link
Member

@ThangHuuVu ThangHuuVu left a comment

Choose a reason for hiding this comment

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

nice, just 1 comment to simplify the default configuration.
btw, could u also update https://github.com/nextauthjs/next-auth/blob/main/docs/pages/data/manifest.json with this new provider? Thanks for the contribution!

Copy link

codecov bot commented Apr 20, 2025

Codecov Report

Attention: Patch coverage is 0% with 234 lines in your changes missing coverage. Please review.

Project coverage is 39.26%. Comparing base (a05451d) to head (66322ae).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
packages/core/src/providers/huggingface.ts 0.00% 234 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12884      +/-   ##
==========================================
- Coverage   39.55%   39.26%   -0.30%     
==========================================
  Files         199      200       +1     
  Lines       31410    31644     +234     
  Branches     1379     1380       +1     
==========================================
  Hits        12425    12425              
- Misses      18985    19219     +234     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ThangHuuVu ThangHuuVu merged commit c1f89ea into nextauthjs:main Apr 20, 2025
11 of 15 checks passed
@maiconcarraro maiconcarraro mentioned this pull request Aug 12, 2025
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Refers to `@auth/core` providers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants