Skip to content
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: Support database session & token refresh for sso providers #6209

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

cy948
Copy link
Contributor

@cy948 cy948 commented Feb 16, 2025 โ€ข

๐Ÿ’ป ๅ˜ๆ›ด็ฑปๅž‹ | Change Type

  • โœจ feat
  • ๐Ÿ› fix
  • โ™ป๏ธ refactor
  • ๐Ÿ’„ style
  • ๐Ÿ‘ท build
  • โšก๏ธ perf
  • ๐Ÿ“ docs
  • ๐Ÿ”จ chore

๐Ÿ”€ ๅ˜ๆ›ด่ฏดๆ˜Ž | Description of Change

๐Ÿ“ ่กฅๅ……ไฟกๆฏ | Additional Information

Copy link

vercel bot commented Feb 16, 2025

@cy948 is attempting to deploy a commit to the LobeHub Team on Vercel.

A member of the Team first needs to authorize it.

@lobehubbot
Copy link
Member

๐Ÿ‘ @cy948

Thank you for raising your pull request and contributing to our Community
Please make sure you have followed our contributing guidelines. We will review it as soon as possible.
If you encounter any problems, please feel free to connect with us.
้žๅธธๆ„Ÿ่ฐขๆ‚จๆๅ‡บๆ‹‰ๅ–่ฏทๆฑ‚ๅนถไธบๆˆ‘ไปฌ็š„็คพๅŒบๅšๅ‡บ่ดก็Œฎ๏ผŒ่ฏท็กฎไฟๆ‚จๅทฒ็ป้ตๅพชไบ†ๆˆ‘ไปฌ็š„่ดก็ŒฎๆŒ‡ๅ—๏ผŒๆˆ‘ไปฌไผšๅฐฝๅฟซๅฎกๆŸฅๅฎƒใ€‚
ๅฆ‚ๆžœๆ‚จ้‡ๅˆฐไปปไฝ•้—ฎ้ข˜๏ผŒ่ฏท้šๆ—ถไธŽๆˆ‘ไปฌ่”็ณปใ€‚

Copy link

codecov bot commented Feb 16, 2025 โ€ข

Codecov Report

All modified and coverable lines are covered by tests โœ…

Project coverage is 97.57%. Comparing base (89b7da0) to head (4f31365).
Report is 66 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6209      +/-   ##
==========================================
+ Coverage   91.53%   97.57%   +6.04%     
==========================================
  Files         689       14     -675     
  Lines       63110     3670   -59440     
  Branches     2951      203    -2748     
==========================================
- Hits        57767     3581   -54186     
+ Misses       5343       89    -5254     
Flag Coverage ฮ”
app ?
server 97.57% <รธ> (+<0.01%) โฌ†๏ธ

Flags with carried forward coverage won't be shown. Click here to find out more.

โ˜” View full report in Codecov by Sentry.
๐Ÿ“ข Have feedback on the report? Share it here.

@cy948 cy948 force-pushed the feat/sso-refresh-token branch from 02179a0 to 50896f0 Compare February 16, 2025 10:15
@cy948 cy948 force-pushed the feat/sso-refresh-token branch from f1646dd to 689c22e Compare February 21, 2025 09:50
@cy948 cy948 changed the title โœจfeat: Support token refresh for sso providers โœจfeat: Support database session & token refresh for sso providers Feb 21, 2025
@cy948 cy948 force-pushed the feat/sso-refresh-token branch 2 times, most recently from bb202f7 to fe8bbb1 Compare February 21, 2025 12:59
@cy948 cy948 force-pushed the feat/sso-refresh-token branch from fe8bbb1 to 4cf2829 Compare February 21, 2025 13:00
@arvinxx
Copy link
Contributor

arvinxx commented Feb 26, 2025 โ€ข

ๅ†™็š„ๅพˆๆฃ’ ๐Ÿ‘ ๅปบ่ฎฎๅฐ†่ฟ™้ƒจๅˆ†ๆ–‡ๆกฃๆ่ฟฐๆๅ‡บๆฅไธ€ไธชไธ“้—จ็š„ RFC~

@lobehubbot
Copy link
Member

Bot detected the issue body's language is not English, translate it automatically. ๐Ÿ‘ฏ๐Ÿ‘ญ๐Ÿป๐Ÿง‘โ€๐Ÿคโ€๐Ÿง‘๐Ÿ‘ซ๐Ÿง‘๐Ÿฟโ€๐Ÿคโ€๐Ÿง‘๐Ÿป๐Ÿ‘ฉ๐Ÿพโ€๐Ÿคโ€๐Ÿ‘จ๐Ÿฟ๐Ÿ‘ฌ๐Ÿฟ


It is very well written, it is recommended to propose a special RFC description in this part of the document~

@cy948
Copy link
Contributor Author

cy948 commented Feb 26, 2025

ๅ†™็š„ๅพˆๆฃ’ ๐Ÿ‘ ๅปบ่ฎฎๅฐ†่ฟ™้ƒจๅˆ†ๆ–‡ๆกฃๆ่ฟฐๆๅ‡บๆฅไธ€ไธชไธ“้—จ็š„ RFC~

ๅฅฝ็š„๏ผŒๆˆ‘็จๅŽๆ•ด็†ไธ€ไธ‹่ต„ๆ–™๏ผŒๅผ€ไธช RFC ๆ่ฟฐไธ€ไธ‹ใ€‚

@lobehubbot
Copy link
Member

Bot detected the issue body's language is not English, translate it automatically. ๐Ÿ‘ฏ๐Ÿ‘ญ๐Ÿป๐Ÿง‘โ€๐Ÿคโ€๐Ÿง‘๐Ÿ‘ซ๐Ÿง‘๐Ÿฟโ€๐Ÿคโ€๐Ÿง‘๐Ÿป๐Ÿ‘ฉ๐Ÿพโ€๐Ÿคโ€๐Ÿ‘จ๐Ÿฟ๐Ÿ‘ฌ๐Ÿฟ


Great writing ๐Ÿ‘ It is recommended to propose a special RFC for description of this part of the document~

OK, I will sort out the information later and open an RFC to describe it.

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.

3 participants