-
Notifications
You must be signed in to change notification settings - Fork 589
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
Correlation filter does not allow any messages through #794
Comments
@ArchiFloyd are these the stesps to repro?
Just to be on the same page, could you share the custom C# code used? |
Thank you for the quick reply.
My hack-ish C#: var filter = new CorrelationRuleFilter { ApplicationProperties = { ["Property1"] = 47381 } };
var ruleOptions = new CreateRuleOptions { Filter = filter };
var topic = "testtopic"
var subscription = "testsub"
var doesTopicExist = await serviceBusAdministrationClient.TopicExistsAsync(
topic,
cancellationToken);
if (doesTopicExist.HasValue
&& doesTopicExist.Value)
{
var doesSubscriptionExist = await serviceBusAdministrationClient.SubscriptionExistsAsync(
topic,
subscription,
cancellationToken);
if (doesSubscriptionExist.HasValue
&& !doesSubscriptionExist.Value)
{
await serviceBusAdministrationClient.CreateSubscriptionAsync(
new CreateSubscriptionOptions(
topic,
subscription)
{
RequiresSession = true
},
ruleOptions ?? new CreateRuleOptions(),
cancellationToken);
}
} |
I suspect the filter is working with a string value rather than numeric. |
In my case I'm not in control of the producer of the messages, thus, cannot change how it behaves. |
Hi!
First and foremost, thank you for the awesome work on service bus explorer. It's a really helpful tool.
I experience that when trying to create a correlation filter marked as the default filter using service bus explorer, no messages are allowed through the filter. If I create the same correlation filter on a subscription through some quick and dirty C# code it allows messages.
The correlation filter I'm trying to create looks like this:
I've observed the issue in the following versions:
5.0.18
6.0.2
I'm not sure what more to include here.
The text was updated successfully, but these errors were encountered: