fix(security): placeholder error mapping in request builders

tomaioo · tomaioo · commit c6d92f736995 · 2026-05-05T17:06:11.000-07:00
All request builder classes use 'XXX' as a placeholder in error_mapping dictionaries: `error_mapping: dict[str, type[ParsableFactory]] = {'XXX': ODataError}`. This placeholder was not replaced during code generation and indicates incomplete error handling for HTTP status codes.

Signed-off-by: tomaioo &lt;203048277+tomaioo@users.noreply.github.com&gt;
diff --git a/msgraph/generated/admin/service_announcement/messages/archive/archive_request_builder.py b/msgraph/generated/admin/service_announcement/messages/archive/archive_request_builder.py
@@ -47,7 +47,8 @@ async def post(self,body: ArchivePostRequestBody, request_configuration: Optiona
         from .....models.o_data_errors.o_data_error import ODataError
 
         error_mapping: dict[str, type[ParsableFactory]] = {
-            "XXX": ODataError,
+            "4XX": ODataError,
+            "5XX": ODataError,
         }
         if not self.request_adapter:
             raise Exception("Http core is null") 

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,8 @@ async def post(self,body: ArchivePostRequestBody, request_configuration: Optiona`
`47`	`47`	`from .....models.o_data_errors.o_data_error import ODataError`
`48`	`48`
`49`	`49`	`error_mapping: dict[str, type[ParsableFactory]] = {`
`50`		`- "XXX": ODataError,`
	`50`	`+ "4XX": ODataError,`
	`51`	`+ "5XX": ODataError,`
`51`	`52`	`}`
`52`	`53`	`if not self.request_adapter:`
`53`	`54`	`raise Exception("Http core is null")`