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

Add horde_stringprep to support SCRAM-SHA-1 #4377

Merged
merged 1 commit into from
Jan 27, 2021
Merged

Conversation

kesselb
Copy link
Contributor

@kesselb kesselb commented Jan 19, 2021

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.

@kesselb kesselb mentioned this pull request Jan 19, 2021
19 tasks
@kesselb kesselb force-pushed the enh/3146/scram-support branch from 4d19faf to cfce705 Compare January 19, 2021 16:31
@@ -13,7 +13,8 @@
"config": {
"platform": {
"php": "7.3"
}
},
"sort-packages": true
Copy link
Member

Choose a reason for hiding this comment

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

😀

@ChristophWurst ChristophWurst modified the milestones: v1.8, v1.9 Jan 20, 2021
@ChristophWurst
Copy link
Member

@kesselb rebase

@kesselb kesselb force-pushed the enh/3146/scram-support branch from cfce705 to 6882d2e Compare January 26, 2021 11:30
@kesselb
Copy link
Contributor Author

kesselb commented Jan 26, 2021

@ChristophWurst done. Small version bump for php-cs-fixer and new github url for webmozart.

@ChristophWurst
Copy link
Member

you rebased away your DCO sign off

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb kesselb force-pushed the enh/3146/scram-support branch from 6882d2e to f747f5e Compare January 26, 2021 13:05
@kesselb kesselb merged commit 2d88317 into master Jan 27, 2021
@kesselb kesselb deleted the enh/3146/scram-support branch January 27, 2021 15:50
@Neustradamus
Copy link

@kesselb: Thanks for SCRAM-SHA-1!

Can you add other SCRAM?

SHA-256/SHA-512/SHA3-512 and -PLUS variants too?

Linked to:

@kesselb
Copy link
Contributor Author

kesselb commented Aug 6, 2023

Hi @Neustradamus,

Can you add other SCRAM?
SHA-256/SHA-512/SHA3-512 and -PLUS variants too?

This depends on Horde ImapClient1 and their peer dependencies.

Footnotes

  1. https://github.com/horde/Imap_Client/blob/e5302ee8b238f03564d0ea4177625214f79a3ebe/lib/Horde/Imap/Client/Auth/Scram.php

@Neustradamus
Copy link

@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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants