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é