Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

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

Closed
Neustradamus opened this issue Jan 10, 2022 · 6 comments

Comments

@Neustradamus
Copy link

Neustradamus commented Jan 10, 2022

Dear @ldapjs team,

In first, I wish you a Happy New Year!

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

"When using the SASL SCRAM mechanism, the SCRAM-SHA-256-PLUS variant SHOULD be preferred over the SCRAM-SHA-256 variant, and SHA-256 variants [RFC7677] SHOULD be preferred over SHA-1 variants [RFC5802]".

https://xmpp.org/extensions/inbox/hash-recommendations.html

-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:

Linked to:

@jsumners
Copy link
Member

What is the action you are expecting with creating this issue? How does this relate to the ldapjs source code?

@Neustradamus
Copy link
Author

It supports SCRAM?

@jsumners
Copy link
Member

It supports SCRAM?

I don't know what you're asking. You have posted a long list of RFC links that we are not inclined to read without cause. Please show that ldapjs has an issue or is missing a feature by supplying a relevant description. If a feature is missing, we would be most receptive to a pull request to add said feature.

@Neustradamus
Copy link
Author

@jsumners: The goal is to have the SCRAM supports in ldapjs.

I have published all informations about SCRAM.

@jsumners
Copy link
Member

Would you like to send a Pull Request to address this issue? Remember to add unit tests.

@jsumners
Copy link
Member

👋

On February 22, 2023, we released version 3 of this library. As a result, we are closing this issue/pull request.

Please see issue #839 for more information, including how to proceed if you feel this closure is in error.

@ldapjs ldapjs locked as resolved and limited conversation to collaborators Feb 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants