-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Python: 11769 - Enabling support for additional extended headers support with application/json #12555
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…mple for patch calls. 2. Changed the order how headers are built by giving preference to what is given in openapi spec responsebody.content
file was mistakenly added
… github.com:sshandilya1991/semantic-kernel into sshandilya/additional_headers_support_openapi_11769
Python Test Coverage Report •
Python Unit Test Overview
|
Hi @sshandilya1991, the pre-commit checks are failing. Please make sure you have them installed locally so they run before committing code. See here for more details: https://github.com/microsoft/semantic-kernel/blob/main/python/DEV_SETUP.md#using-uv.
|
… github.com:sshandilya1991/semantic-kernel into sshandilya/additional_headers_support_openapi_11769
Hi @sshandilya1991, the pre-commit check that is failing is related to the uv.lock check. If you delete your local uv file, and then re-run |
Head branch was pushed to by a user without write access
Hi @sshandilya1991, please make sure you have the pre-commit installed, so the necessary pre-commit steps are run before committing code. The ruff pre-commit check is failing. It will make a local modification, which you can then |
Hi @sshandilya1991, please re-open the PR when you're able to install the pre-commit and fix the checks. Thanks. |
Enhanced the OpenAPI plugin's media type validation by implementing regex pattern matching and improved Content-Type header handling for better flexibility and operation support - primarily resolving #11769
Key Changes
is_supported_media_type()
method with comprehensive validation logic