Resolve "Allow matching of social accounts to local accounts by email" #3118

Closed
hansegucker wants to merge 4 commits from 1021-allow-matching-of-social-accounts-to-local-accounts-by-email into master
Owner

Closes #1021

Closes #1021
Author
Owner

added 3 commits

  • c95891a5 - Create EmailAddress objects for usage with social account
  • a0c0760c - Activate EMAIL_AUTHENTICATION for social accounts
  • fffd13f0 - [Docs] Update outdated instructions on social accounts

Compare with previous version

added 3 commits <ul><li>c95891a5 - Create EmailAddress objects for usage with social account</li><li>a0c0760c - Activate EMAIL_AUTHENTICATION for social accounts</li><li>fffd13f0 - [Docs] Update outdated instructions on social accounts</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-Core/-/merge_requests/1473/diffs?diff_id=43402&start_sha=7cfbec3c651254fa8c0e1440bb9265bfa915c621)
Author
Owner

marked this merge request as ready

marked this merge request as **ready**
Author
Owner

assigned to @nik

assigned to @nik
Member
  settings = { server_url =  'https://example.org' }
```suggestion:-0+0 settings = { server_url = 'https://example.org' } ```
Member

resolved all threads

resolved all threads
Member

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](/AlekSIS/official/AlekSIS-Core/-/merge_requests/1473/diffs?diff_id=43430&start_sha=fffd13f0a29fd83652a4197ee08446b147ea3f48#6df655219a6dd05bc05767fe193daa8943c53425_40_40)
Member

added 1 commit

  • 04ae16be - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

added 1 commit <ul><li>04ae16be - Apply 1 suggestion(s) to 1 file(s)</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-Core/-/merge_requests/1473/diffs?diff_id=43430&start_sha=fffd13f0a29fd83652a4197ee08446b147ea3f48)
Owner

Rejected.

E-mail addresses are not an identifier, and the OIDC standard even forbids relying on them for linking accounts.

Rejected. E-mail addresses are not an identifier, and the OIDC standard even forbids relying on them for linking accounts.
nik closed this pull request 2024-06-04 14:29:04 +02:00

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
4 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
aleksis/AlekSIS-Core!3118
No description provided.