Skip to content

fix(authz): unauthenticated error code for deleted service account #10878

Merged
vikrantgupta25 merged 6 commits intomainfrom
fix/error-deleted-sa
Apr 13, 2026
Merged

fix(authz): unauthenticated error code for deleted service account #10878
vikrantgupta25 merged 6 commits intomainfrom
fix/error-deleted-sa

Conversation

@vikrantgupta25
Copy link
Copy Markdown
Member

@vikrantgupta25 vikrantgupta25 commented Apr 8, 2026

📄 Summary

  • unauthenticated error code for deleted service account
  • make the service account names and factor api key names accept more relaxed regexp.

contributes to: https://github.com/SigNoz/platform-pod/issues/1830

@vikrantgupta25 vikrantgupta25 added the safe-to-integrate Run integration tests label Apr 8, 2026
@github-actions github-actions Bot added the bug Something isn't working label Apr 8, 2026
@vikrantgupta25 vikrantgupta25 added safe-to-integrate Run integration tests and removed safe-to-integrate Run integration tests labels Apr 8, 2026
@vikrantgupta25 vikrantgupta25 marked this pull request as ready for review April 13, 2026 07:54
@vikrantgupta25 vikrantgupta25 added safe-to-integrate Run integration tests and removed safe-to-integrate Run integration tests labels Apr 13, 2026
@vikrantgupta25 vikrantgupta25 added safe-to-integrate Run integration tests and removed safe-to-integrate Run integration tests labels Apr 13, 2026
Comment thread pkg/modules/serviceaccount/implserviceaccount/store.go Outdated
Comment thread pkg/modules/serviceaccount/implserviceaccount/module.go Outdated
@vikrantgupta25 vikrantgupta25 added safe-to-integrate Run integration tests and removed safe-to-integrate Run integration tests labels Apr 13, 2026
@vikrantgupta25 vikrantgupta25 added this pull request to the merge queue Apr 13, 2026
Merged via the queue into main with commit 9ec6da5 Apr 13, 2026
152 of 153 checks passed
@vikrantgupta25 vikrantgupta25 deleted the fix/error-deleted-sa branch April 13, 2026 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working safe-to-integrate Run integration tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants