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

[ENH] Adding SCRAM-SHA authentication mechanisms to cypht #1072

Merged
merged 1 commit into from
Jun 13, 2024
Merged

Conversation

Danelif
Copy link
Member

@Danelif Danelif commented Jun 7, 2024

In this PR I implement SCRAM-SHA-1/512 and SCRAM-SHA-1/512-PLUS authentication mechanisms into cypht

Related issue

Related issue

@marclaporte
Copy link
Member

@Neustradamus please advise.

@Danelif Danelif changed the title [EHN] Adding SCRAM-256-PLUS authentication mechanism to cypht [EHN] Adding SCRAM-SHA-256-PLUS authentication mechanism to cypht Jun 7, 2024
@Danelif Danelif force-pushed the enh-scram branch 2 times, most recently from bc2c3a9 to e9b85d4 Compare June 7, 2024 14:08
@Danelif Danelif changed the title [EHN] Adding SCRAM-SHA-256-PLUS authentication mechanism to cypht [EHN] Adding SCRAM-SHA-256, SCRAM-SHA-256-PLUS, SCRAM-SHA-1, SCRAM-SHA-1-PLUS authentication mechanisms to cypht Jun 7, 2024
@Danelif
Copy link
Member Author

Danelif commented Jun 7, 2024

i just finished adding scram-sha-256 version to the code

@Danelif Danelif changed the title [EHN] Adding SCRAM-SHA-256, SCRAM-SHA-256-PLUS, SCRAM-SHA-1, SCRAM-SHA-1-PLUS authentication mechanisms to cypht [ENH] Adding SCRAM-SHA-256, SCRAM-SHA-256-PLUS, SCRAM-SHA-1, SCRAM-SHA-1-PLUS authentication mechanisms to cypht Jun 7, 2024
@Danelif Danelif force-pushed the enh-scram branch 2 times, most recently from 64c9d01 to d6d6561 Compare June 10, 2024 08:10
@Danelif Danelif changed the title [ENH] Adding SCRAM-SHA-256, SCRAM-SHA-256-PLUS, SCRAM-SHA-1, SCRAM-SHA-1-PLUS authentication mechanisms to cypht [ENH] Adding SCRAM-SHA authentication mechanisms to cypht Jun 10, 2024
@Danelif Danelif assigned kroky and unassigned kroky Jun 10, 2024
@Danelif Danelif requested review from kroky and josaphatim June 10, 2024 15:13
modules/imap/hm-imap.php Outdated Show resolved Hide resolved
modules/imap/hm-imap.php Outdated Show resolved Hide resolved
modules/smtp/hm-smtp.php Outdated Show resolved Hide resolved
@kroky
Copy link
Member

kroky commented Jun 11, 2024

Great you added unit tests!

@Danelif Danelif requested review from kroky and removed request for josaphatim June 12, 2024 11:07
modules/scram/scram.php Outdated Show resolved Hide resolved
@kroky kroky merged commit cdeee04 into master Jun 13, 2024
6 checks passed
@marclaporte marclaporte deleted the enh-scram branch June 13, 2024 11:43
@Neustradamus
Copy link

@Danelif: Good job!

@Neustradamus
Copy link

About SHA3-512/SHA3-512-PLUS, it is not possible to add too?

@Danelif
Copy link
Member Author

Danelif commented Jun 16, 2024

@Neustradamus It is possible to add SHA3-512/SHA3-512-PLUS.

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.

None yet

4 participants