You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/09/19 01:47:01 UTC

[GitHub] [pulsar] codelipenghui commented on pull request #17151: [fix][broker] Added check for invisible characters for subscription name

codelipenghui commented on PR #17151:
URL: https://github.com/apache/pulsar/pull/17151#issuecomment-1250461038

   > I think the change makes sense. Is there a reason we don't do this validation for subscriptions created via auto subscription creation in the Pulsar Protocol?
   
   We should add the validation. It's better to keep consistent with the REST API and Pulsar Protocol.


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org