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.

Sign up for GitHub

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

Jump to bottom

Azure Service Bus Queue and Azure Queue Storage registrations use the same name #665

Closed
adrianiftode opened this issue Nov 7, 2020 · 2 comments · Fixed by #2038
Closed

Azure Service Bus Queue and Azure Queue Storage registrations use the same name #665

adrianiftode opened this issue Nov 7, 2020 · 2 comments · Fixed by #2038
Assignees
@adamsitnik
Labels
bug Something isn't working

Comments

@adrianiftode
Copy link

adrianiftode commented Nov 7, 2020 •
edited
Loading

Please, fill the following sections to help us fix the issue

What happened:

These two health checks use the same name for the corresponding registrations

What you expected to happen:

To use different names

How to reproduce it (as minimally and precisely as possible):

AddAzureQueueStorage

AddAzureServiceBusQueue
@CarlosLanderas CarlosLanderas added the bug Something isn't working label Nov 22, 2020
@CarlosLanderas
Copy link
Contributor

@adrianiftode this looks like a bug!. Thanks for catching this

@adamsitnik adamsitnik mentioned this issue Sep 19, 2023
Merged
6 tasks
@adamsitnik adamsitnik self-assigned this Sep 19, 2023
@sungam3r
Copy link
Collaborator

sungam3r commented Dec 3, 2023

Names are azure_queue_storage and azurequeue after #2038

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

@adamsitnik adamsitnik

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

split Azure packages into dedicated nuget packages adamsitnik/AspNetCore.Diagnostics.HealthChecks
4 participants
@adamsitnik @adrianiftode @CarlosLanderas @sungam3r