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

Async NATS client does not support inbox prefix (and should) #602

Closed
autodidaddict opened this issue Aug 9, 2022 · 1 comment · Fixed by #604
Closed

Async NATS client does not support inbox prefix (and should) #602

autodidaddict opened this issue Aug 9, 2022 · 1 comment · Fixed by #604
Assignees
Labels
bug Confirmed reproducible bug enhancement Enhancement to existing functionality

Comments

@autodidaddict
Copy link
Collaborator

From the documentation in a number of places, it states that NATS clients support the ability to define an inbox prefix:

However, as a client, there is the option of defining an explicit inbox prefix other than _INBOX.

But when looking at the async nats library, there is no way to generate an inbox with a prefix of anything other than _INBOX, which you can see here: https://docs.rs/async-nats/latest/src/async_nats/client.rs.html#193

I would like to be able to have a function in the client builder like inbox_prefix() that, when used, sets an alternate prefix other than _INBOX. There's no real workaround to this because the request functions use new_inbox() internally.

@autodidaddict autodidaddict added the bug Confirmed reproducible bug label Aug 9, 2022
@Jarema
Copy link
Member

Jarema commented Aug 9, 2022

Thanks for filling the issue!

We will consider what's the best way to allow for custom inboxes and implement a proper solution.

@Jarema Jarema self-assigned this Aug 9, 2022
@Jarema Jarema added the enhancement Enhancement to existing functionality label Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed reproducible bug enhancement Enhancement to existing functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants