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

MqttConfig.java does not handle both a subscribe and publish under the same topic #414

Open
gaby-roland opened this issue Jul 18, 2024 · 3 comments
Labels

Comments

@gaby-roland
Copy link

gaby-roland commented Jul 18, 2024

Having a publish and subscribe channels under the same topic is allowed, however, it results in compilation errors due to the subscribe topic variables not being created. This is because the loop iterates through the channels, and uses if/elif to create the publish/subscribe topic variables. If both publish and subscribe are defined in the yaml, only the publish variables are created in the Config.java class. This is an easy fix, just change the if/elif to if/if to handle this edge case.

{% elif channel.hasSubscribe() %}

Copy link

Welcome to AsyncAPI. Thanks a lot for reporting your first issue. Please check out our contributors guide and the instructions about a basic recommended setup useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

@Tenischev
Copy link
Member

Tenischev commented Jul 18, 2024

Hello @gaby-roland
Thank you for the report!
Could you please also provide an example of AsyncAPI file with this configuration? If possible, could you please explain application logic that you expect from application that read and write to the same topic?

Copy link

This issue has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this issue, add a comment with a detailed explanation.

There can be many reasons why some specific issue has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this issue forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added the stale label Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants