GH-8720: Check MQTT topics if not empty strings #8722
Conversation
|
I'll back-port to |
| * @since 4.1 | ||
| */ | ||
| @ManagedOperation | ||
| public void addTopic(String... topic) { |
There was a problem hiding this comment.
For consistency use the plural naming: topic -> topics.
| * @since 4.1 | ||
| */ | ||
| @ManagedOperation | ||
| public void addTopics(String[] topic, int[] qos) { |
There was a problem hiding this comment.
Same, for consistency use the plural naming: topic -> topics.
| if (topic == null && this.defaultTopic == null) { | ||
| throw new IllegalStateException( | ||
| "No topic could be determined from the message and no default topic defined"); | ||
| if (topic == null) { |
There was a problem hiding this comment.
Can the topic be returned as an empty string?
There was a problem hiding this comment.
The MQTT client validates the topic before publishing.
And that is done in a sync mode, so even if it is returned here as empty, it is going to be reject immediately in that publish() implementations.
Fixes spring-projects#8720 Validate MQTT topics for empty strings in the channel adapters configuration **Cherry-pick to `6.1.x` & `6.0.x`**
|
Back-ported to |
2 participants
Fixes #8720
Validate MQTT topics for empty strings in the channel adapters configuration
Cherry-pick to
6.1.x&6.0.x