Open AnandPaladin opened 1 month ago
The modifications in ErrorManageUtil.java
enrich the triggerNotificationPermissionDenied
method by adding new exception strings ("Unauthorized"
, "Forbidden"
, and "ForbiddenByRbac"
) to the exceptionList
. This enhancement helps in better error handling and notification when specific permission issues occur in the Azure inventory utility.
File Path | Change Summary |
---|---|
...bile/pacbot/azure/inventory/util/ErrorManageUtil.java |
Updated triggerNotificationPermissionDenied() to include additional exceptions in the exceptionList and misconfiguredExceptionList lists. |
sequenceDiagram
participant User
participant ErrorManageUtil
Note over User,ErrorManageUtil: Permission Denied Scenario
User ->> ErrorManageUtil: triggerNotificationPermissionDenied
ErrorManageUtil ->> ErrorManageUtil: Check exceptions (Unauthorized, Forbidden, ForbiddenByRbac)
ErrorManageUtil -->> User: Notify Permission Denied
🐇 In the realm of Azure's might,
New exceptions come to light.
Unauthorized, Forbidden's call,
ErrorManageUtil
catches them all.
Permissions clear, the path defined,
In code's embrace, solutions find. 🚀
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
Description
This is related to https://paladincloud.atlassian.net/browse/CORE-2050 where vertex azure collector throwing datadog alerts due to "UnAuthorized" exception such as 401/403 error codes. These error codes should be excluded from datadog alerts because we are triggering notification for those alerts.
Problem
Some of the error codes are not in exclusion list.Detailed root causes are documented in this https://docs.google.com/spreadsheets/d/1dKZwK_bFE_4YRjyQX91vwYYdSxqSZEdVuyDeeZXxSb0/edit#gid=0