-
Notifications
You must be signed in to change notification settings - Fork 262
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
Add horde_stringprep to support SCRAM-SHA-1 #4377
Conversation
4d19faf
to
cfce705
Compare
@@ -13,7 +13,8 @@ | |||
"config": { | |||
"platform": { | |||
"php": "7.3" | |||
} | |||
}, | |||
"sort-packages": true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😀
@kesselb rebase |
cfce705
to
6882d2e
Compare
@ChristophWurst done. Small version bump for php-cs-fixer and new github url for webmozart. |
you rebased away your DCO sign off |
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
6882d2e
to
f747f5e
Compare
@kesselb: Thanks for SCRAM-SHA-1! Can you add other SCRAM? SHA-256/SHA-512/SHA3-512 and -PLUS variants too? Linked to: |
Hi @Neustradamus,
This depends on Horde ImapClient1 and their peer dependencies. Footnotes |
@kesselb: I wish you a Happy New Year 2024! Recently, there have been a lot of updates in PEAR about SCRAM. It is possible to look for Horde and Nextcloud? |
Related to #3146
Horde_Imap_Client supports the SCRAM authentication mechanism but Horde_Stringprep as dependency is required (https://github.com/horde/Imap_Client/blob/e5302ee8b238f03564d0ea4177625214f79a3ebe/lib/Horde/Imap/Client/Auth/Scram.php#L98-L101).
If the server supports SCRAM-SHA-1 and the connection is unencrypted we are now able to use it.