loafle/openapi-generator-original
3
0
Fork 1
mirror of https://github.com/OpenAPITools/openapi-generator.git synced 2025-05-12 20:50:55 +00:00
Code Issues Packages Projects Releases Wiki Activity

[php-symfony] Never return 406 when user accepts */*

Browse Source 
When a query has header "Accept" set to "*/*" it means it accepts
everything. It is hence weird to return a 406.
This patch ensures it does not occur: when the query accepts everything
then we take any produced type.

This fixes #13334. This also partly makes the open PR #15560 obsolete
(or at least, it provides a workaround)
This commit is contained in:
Guillaume Turri 2025-05-09 21:02:30 +02:00
parent d6c4634269
commit 7b7c15ce6c
2 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
modules/openapi-generator/src/main/resources/php-symfony/Controller.mustache vendored
View File

@ -204,6 +204,10 @@ class Controller extends AbstractController
return 'application/xml'; return 'application/xml';
} }
if (in_array('*/*', $accept)) {
return $produced[0];
}
// If we reach this point, we don't have a common ground between server and client // If we reach this point, we don't have a common ground between server and client
return null; return null;
} }

4
samples/server/petstore/php-symfony/SymfonyBundle-php/Controller/Controller.php
View File

@ -214,6 +214,10 @@ class Controller extends AbstractController
return 'application/xml'; return 'application/xml';
} }
if (in_array('*/*', $accept)) {
return $produced[0];
}
// If we reach this point, we don't have a common ground between server and client // If we reach this point, we don't have a common ground between server and client
return null; return null;
} }