rueian opened a new pull request, #1406: URL: https://github.com/apache/incubator-kvrocks/pull/1406
Hi, following the conversation https://github.com/apache/incubator-kvrocks/issues/1403 This PR makes the `HELLO` command: 1. accept the `default` username as well. 2. use case insensitive check on the `AUTH` and `SETNAME`. Clients may send these options in lower or upper cases. For example, go-redis sends them in lower cases. I think this PR won't break any existing Kvrocks clients but will remove barriers for other clients to adapt Kvrocks by aligning HELLO syntax with Redis. Please let me know if these changes make sense. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
