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

how to use rabbitmq as messagebus or queue? #1526

Open
sampsonye opened this issue Mar 24, 2024 · 9 comments
Open

how to use rabbitmq as messagebus or queue? #1526

sampsonye opened this issue Mar 24, 2024 · 9 comments
Assignees

Comments

@sampsonye
Copy link

Foundatio.Rabbitmq use uri-spec as connectionstring (https://www.rabbitmq.com/docs/uri-spec)

but in Exceptionless, we can not build it like uri

image

can give me an example?

@niemyjski
Copy link
Member

We probably need to take a pass at redoing all of configuration to be more explicit, but you should be able to do this two ways.

  1. Set the MessageBus connection string to provider=rabbitmq;CONNECTION_STRING_HERE
  2. Set the MessageBus connection string to provider=rabbitmq; then add rabbitmq to the connection strings app config / env section with the raw connection string

cc @ejsmith

@sampsonye
Copy link
Author

We probably need to take a pass at redoing all of configuration to be more explicit, but you should be able to do this two ways.

  1. Set the MessageBus connection string to provider=rabbitmq;CONNECTION_STRING_HERE
  2. Set the MessageBus connection string to ; then add rabbitmq to the connection strings app config / env section with the raw connection stringprovider=rabbitmq

cc @ejsmith
all dosn't work fine because exceptionless request connectionstring must like key=value patten
image
ParseConnectionString() and BuildConnectionString() method working on it.

@niemyjski
Copy link
Member

Can you please add a sample connection string that can be bested. If you wouldn't mind adding a test and or updating this that would be a huge help. I know the cache one supports this, might have just been a miss.

@sampsonye
Copy link
Author

image

@sampsonye
Copy link
Author

image

@niemyjski
Copy link
Member

Can you paste that connection string here and also try escaping it with quotes?

@niemyjski
Copy link
Member

I was finally able to dig in and get this fixed. I've submitted a pr for the team to review.

@niemyjski
Copy link
Member

Thanks for logging this, this is fixed in a nightly build

niemyjski added a commit that referenced this issue Apr 7, 2024
@niemyjski niemyjski reopened this Apr 7, 2024
@niemyjski
Copy link
Member

I had to revert this change as it was a pretty big breaking change to configuration. We will address this shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants