Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Topics with the same name but different message types #32257

Merged

Conversation

johanandren
Copy link
Member

@johanandren johanandren commented Dec 13, 2023

Should not publish messages of the wrong type to subscribers

Should not publish messages of the wrong type to subscribers
Copy link
Member

@patriknw patriknw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@johanandren johanandren merged commit 764a9da into akka:main Dec 13, 2023
5 checks passed
@johanandren johanandren deleted the wip-topics-with-different-message-types branch December 13, 2023 09:05
@johanandren johanandren added this to the 2.9.1 milestone Dec 13, 2023
@johanandren
Copy link
Member Author

This does not fix the fact that separate message type topics with the same name will get a message sent over the wire though, so we should probably still update the docs to not say topic name + message type is unique and recommend unique topic names.

He-Pin pushed a commit to He-Pin/akka that referenced this pull request Jan 7, 2024
Should not publish messages of the wrong type to subscribers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants