You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "pdolif (via GitHub)" <gi...@apache.org> on 2024/12/08 08:30:05 UTC

[I] [Bug] TopicName does not throw IllegalArgumentException if localName is whitespace only [pulsar]

pdolif opened a new issue, #23690:
URL: https://github.com/apache/pulsar/issues/23690

   ### Search before asking
   
   - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar.
   
   
   ### Read release policy
   
   - [X] I understand that unsupported versions don't get bug fixes. I will attempt to reproduce the issue on a supported version of Pulsar client and Pulsar broker.
   
   
   ### Version
   
   Pulsar master
   
   ### Minimal reproduce step
   
   Call `TopicName.get(" ")`.
   
   ### What did you expect to see?
   
   This is a test for the behavior I would expect:
   ```java
   try {
       TopicName.get(" ");
       fail("Should have raised exception");
   } catch (IllegalArgumentException e) {
       // Ok
   }
   ```
   
   When creating a consumer with the topic name only being a whitespace we get:
   
   > java.lang.IllegalArgumentException: topicNames cannot have blank topic
   
   ### What did you see instead?
   
   No exception is thrown.
   
   ### Anything else?
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [X] I'm willing to submit a PR!


-- 
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.apache.org

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


Re: [I] [Bug] TopicName does not throw IllegalArgumentException if localName is whitespace only [pulsar]

Posted by "Technoboy- (via GitHub)" <gi...@apache.org>.
Technoboy- closed issue #23690: [Bug] TopicName does not throw IllegalArgumentException if localName is whitespace only
URL: https://github.com/apache/pulsar/issues/23690


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