Hi Proven,

On Tue, Feb 21, 2023 at 1:37 PM Proven Provenzano
<pprovenz...@confluent.io.invalid> wrote:
>
> Hi Jose,
>
> 1. The SCRAM in SCRAM-SHA-256 is required as the mechanism name is
> SCRAM-SHA-256.
>     I do realize there is a bit of redundancy here.
>
> 2. I'll add documentation for all the possible values. They are
> SCRAM-SHA-256 and SCRAM-SHA-512.
>
> 3. I'd like to keep it with a capital letter as it is a specific type of
> message we want to insert. I do agree that -A
>     is not the correct choice so I'll suggest -S for SCRAM and that leaves
> -A for --add-acl and -a for a generic
>     --add-config in the future.
>

It makes sense to me. Thanks for the changes.

-- 
-José

Reply via email to