-
Notifications
You must be signed in to change notification settings - Fork 985
Closed
Labels
enterpriseEnterprise-license / premium functionalityEnterprise-license / premium functionalitys4Internal bugs (e.g. test flakes), extreme edge cases, and bug risksInternal bugs (e.g. test flakes), extreme edge cases, and bug riskssiteArea: frontend dashboardArea: frontend dashboard
Description
the deployment stats footer displays if the API call returns a 200, and is hidden if it returns a non-200 code. this results in every unauthorized user (non Owners) inadvertently spamming the logs with 400s, because they are (correctly) unauthorized to view the /deployment/stats
and /debug/health
routes.
this is causing log noise for our customers.
this should be added to the /api/v2/authcheck
list and conditionally call viewDeploymentStats
here:
coder/site/src/components/AuthProvider/permissions.tsx
Lines 84 to 88 in 9f3a955
[checks.viewDeploymentStats]: { | |
object: { | |
resource_type: "deployment_stats", | |
}, | |
action: "read", |
matifali
Metadata
Metadata
Assignees
Labels
enterpriseEnterprise-license / premium functionalityEnterprise-license / premium functionalitys4Internal bugs (e.g. test flakes), extreme edge cases, and bug risksInternal bugs (e.g. test flakes), extreme edge cases, and bug riskssiteArea: frontend dashboardArea: frontend dashboard