Skip to content

feat: added getEmailVerified() and isEmailTrustworthy()#132

Merged
shadowhand merged 1 commit into
thephpleague:mainfrom
dt-thomas-durand:feat/email-verified-or-trustworthy
Dec 15, 2025
Merged

feat: added getEmailVerified() and isEmailTrustworthy()#132
shadowhand merged 1 commit into
thephpleague:mainfrom
dt-thomas-durand:feat/email-verified-or-trustworthy

Conversation

@dt-thomas-durand

Copy link
Copy Markdown
Contributor

Based on Google documentation, we can expose if the email was verified, but also some logic to know if the email can be used for authentication purposes, or not.

See relevant documentation https://developers.google.com/identity/gsi/web/guides/verify-google-id-token

Comment thread src/Provider/GoogleUser.php Outdated
Comment thread src/Provider/GoogleUser.php Outdated
@dt-thomas-durand
dt-thomas-durand force-pushed the feat/email-verified-or-trustworthy branch from 5982cd6 to 979d50b Compare December 15, 2025 11:27
@shadowhand

Copy link
Copy Markdown
Member

@dt-thomas-durand will you please rebase? I just corrected the CodeCov configuration, which should allow the 8.2, Code coverage test to pass.

@dt-thomas-durand
dt-thomas-durand force-pushed the feat/email-verified-or-trustworthy branch from 412a05c to bd0a3d1 Compare December 15, 2025 12:15
@dt-thomas-durand

Copy link
Copy Markdown
Contributor Author

@dt-thomas-durand will you please rebase? I just corrected the CodeCov configuration, which should allow the 8.2, Code coverage test to pass.

I just did!

@codecov

codecov Bot commented Dec 15, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (5b784be) to head (bd0a3d1).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##                main      #132   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity        34        40    +6     
===========================================
  Files              3         3           
  Lines             71        82   +11     
===========================================
+ Hits              71        82   +11     

☔ 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.

@shadowhand

Copy link
Copy Markdown
Member

✅ tests are good, thank you!

@shadowhand
shadowhand merged commit 158be45 into thephpleague:main Dec 15, 2025
11 checks passed
@dt-thomas-durand

Copy link
Copy Markdown
Contributor Author

✅ tests are good, thank you!

Pleasure is mine!

@shadowhand

Copy link
Copy Markdown
Member

Released in version 4.1.0.

@dt-thomas-durand

Copy link
Copy Markdown
Contributor Author

Released in version 4.1.0.

Damn, you're Lucky Luke of the open source!
Thank you so much!

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.

2 participants