-
Notifications
You must be signed in to change notification settings - Fork 333
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
Aspire default table HealthCheck fails with 400 bad request #2450
Comments
FYI This is still happening with .NET 9, latest aspire |
@EmmaZhu @onionhammer |
This is the only relevant log the container spits out: 2024-11-15T08:24:11 172.17.0.1 - - [15/Nov/2024:14:24:11 +0000] "GET /devstoreaccount1/Tables?$format=application%2Fjson%3Bodata%3Dminimalmetadata&$filter=false HTTP/1.1" 400 - |
I have the same issue. Looks like this call from in health check class causes the exception: Health check source code: So, azurite don't like such query filter. |
The workaround is to disable built-in health check:
and add your own health check. |
Whatever is emitting that query is forgetting to put () after Tables. I'm not sure but I think this may be a problem with the Azure table client? |
Which service(blob, file, queue, table) does this issue concern?
Table
Which version of the Azurite was used?
3.31.0
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
DockerHub
What's the Node.js version?
NA -- Came w/ docker image
What problem was encountered?
More stack:
Health check source code:
https://github.com/Xabaril/AspNetCore.Diagnostics.HealthChecks/blob/master/src/HealthChecks.Azure.Data.Tables/AzureTableServiceHealthCheck.cs
Steps to reproduce the issue?
After switching to Azurite, using the default health checker that comes with the Aspire component for adding azure table storage, the health check is receiving a 400 bad request.
Have you found a mitigation/solution?
Not yet
The text was updated successfully, but these errors were encountered: