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

SCRAM-SHA-1(-PLUS) + SCRAM-SHA-256(-PLUS) + SCRAM-SHA-512(-PLUS) + SCRAM-SHA3-512(-PLUS) supports #9

Closed
Neustradamus opened this issue Nov 4, 2020 · 7 comments

Comments

@Neustradamus
Copy link

Neustradamus commented Nov 4, 2020

Dear @gbarr,

Can you add supports of :

  • SCRAM-SHA-1
  • SCRAM-SHA-1-PLUS
  • SCRAM-SHA-256
  • SCRAM-SHA-256-PLUS
  • SCRAM-SHA-512
  • SCRAM-SHA-512-PLUS
  • SCRAM-SHA3-512
  • SCRAM-SHA3-512-PLUS

You can add too:

  • SCRAM-SHA-224
  • SCRAM-SHA-224-PLUS
  • SCRAM-SHA-384
  • SCRAM-SHA-384-PLUS

SCRAM-SHA-1(-PLUS):

SCRAM-SHA-256(-PLUS):

SCRAM-SHA-512(-PLUS):

SCRAM-SHA3-512(-PLUS):

-PLUS variants:

IMAP:

LDAP:

  • RFC5803: Lightweight Directory Access Protocol (LDAP) Schema for Storing Salted: Challenge Response Authentication Mechanism (SCRAM) Secrets: https://tools.ietf.org/html/rfc5803

HTTP:

2FA:

IANA:

Note, after SCRAM-SHA-1(-PLUS):

Linked to:

@ehuelsmann
Copy link
Collaborator

@Neustradamus No need to implement this in Authen::SASL. My dist implements the non -PLUS variants: https://metacpan.org/pod/Authen::SASL::SCRAM

@ehuelsmann
Copy link
Collaborator

@Neustradamus there's no RFC describing 224 or 384, right? Why then would anybody want to implement that? As for SHA3-512, that's in draft state. However, none of that is related to this repository. The repository for Authen::SASL::SCRAM is https://github.com/ehuelsmann/authen-sasl-scram.

@Neustradamus
Copy link
Author

@ehuelsmann: It is based on SHA-1 RFC and it has been in https://metacpan.org/pod/Authen::SCRAM.
To move people from Authen::SCRAM it is really important too.
SCRAM-SHA-512 is like SCRAM-SHA3-512, already supported by some projects...

@ehuelsmann
Copy link
Collaborator

When I said that none of this matters to this repository while referring you to the Authen::SASL::SCRAM one, I meant that the discussion - if any - should be continued there. I'll do so in a minute.

@Neustradamus
Copy link
Author

@ehuelsmann: I am sorry, can you delete the text from your ticket and close it?
I will do a ticket in your repo.

@Neustradamus
Copy link
Author

The ticket has been updated like all others.

@ehuelsmann
Copy link
Collaborator

Since Authen::SASL::SCRAM is about implementing SCRAM for SASL, this is not the place for this ticket.

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

No branches or pull requests

2 participants