Skip to content

.Net: New Feature: Supporting additional content-type headers / media types for api calls. #11769

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

Open
beyondszine opened this issue Apr 28, 2025 · 1 comment
Assignees
Labels
.NET Issue or Pull requests regarding .NET code python Pull requests for the Python Semantic Kernel

Comments

@beyondszine
Copy link

beyondszine commented Apr 28, 2025


name: Feature request
about: Supporting addtional content-type headers / media types for api calls.


what's the need

Http requests/responses which contains header other than the allowed ones currently referenced here-dotnot & here-python
are not allowed & give parsing error. Idea is to enable support for additional headers which belong to same category but differ due to RFC's flexibility to support for more specific and informative header types.

Sample cases:

Major Syntax Some Examples
JSON application/json, application/problem+json, application/ld+json, application/json-patch+json, application/merge-patch+json
XML application/xml, application/atom+xml, application/soap+xml, application/rdf+xml
Text text/plain, text/html, text/csv, text/calendar, text/markdown
Multipart multipart/form-data, multipart/related, multipart/mixed
Vendor Specific application/vnd.api+json, application/vnd.github+json, application/vnd.ms-excel

To being with I'd suggest to focus only on JSON based extensions on the format of application/<something>+json referenced in following RFCs:

I can help with the PR for the above but was wondering to know the plan and/or thoughts of maintainers on this & if a common thought is there - happy to work on this.

EDIT: I mistakenly said above that currently it gives parsing errors. I see that it's conciously ignored from a PR. Seems like a thought through step for the time being.

@markwallace-microsoft markwallace-microsoft added .NET Issue or Pull requests regarding .NET code python Pull requests for the Python Semantic Kernel triage labels Apr 28, 2025
@github-actions github-actions bot changed the title New Feature: Python: New Feature: Apr 28, 2025
@github-actions github-actions bot changed the title Python: New Feature: .Net: New Feature: Apr 28, 2025
@markwallace-microsoft markwallace-microsoft changed the title .Net: New Feature: .Net: New Feature: Supporting addtional content-type headers / media types for api calls. Apr 28, 2025
@markwallace-microsoft markwallace-microsoft changed the title .Net: New Feature: Supporting addtional content-type headers / media types for api calls. .Net: New Feature: Supporting additional content-type headers / media types for api calls. Apr 28, 2025
@SergeyMenshykh
Copy link
Member

Hi @beyondszine, thanks for volunteering to add support for the application/*+json types. These and others are not yet supported because no one has asked for them. So, please go ahead and open a PR, or ideally, a few small ones to simplify the review process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
.NET Issue or Pull requests regarding .NET code python Pull requests for the Python Semantic Kernel
Projects
None yet
Development

No branches or pull requests

3 participants