Should we consider ce-datacontenttype header when no Content-Type HTTP header is informed?
See original GitHub issueHi!
Just a quick question. I see that we are considering the HTTP header Content-type
as the ce-datacontenttype
attribute. It’s a good thing, but I came up with certain scenarios where the message comes without the Content-type
header, but only with ce-datacontenttype
. In such scenarios, we might fail to serialize the message correctly.
Issue Analytics
- State:
- Created 2 years ago
- Comments:5 (3 by maintainers)
Top Results From Across the Web
Why do we need to set the Content-Type header of an HTTP ...
The "Content-Type" header field indicates the media type of the associated representation: either the representation enclosed in the message ...
Read more >Content-Type - HTTP - MDN Web Docs - Mozilla
The Content-Type representation header is used to indicate the original media type of the resource (prior to any content encoding applied ...
Read more >Missing Content-Type Header Detected on Web Application
A Missing Content-Type Header is an attack that is similar to a Code Evaluation (RoR) that -level severity. Categorized as a CWE-16, ISO27001-A.14.1.2, ......
Read more >What is the Content-Type http header for? - MIT Media Lab
The Content-Type http request header specifies the content type of the http request payload. The Content-Type header is NOT tied to the content...
Read more >Content-Type HTTP Header: Syntax, Directive, Examples
The Content-Type HTTP Header is used to display the asset's first media type before any substance encoding is used for transmission.
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
I don’t think it’s related to this SDK. I understand that the Knative Broker is not forwarding the
content-type
correctly when the message is acloudevents+json
. Although it is correctly parsing the CE. I put this gist together with a reproducer: https://gist.github.com/ricardozanini/9d0652194db6738245fa4fcc566b543eI think I’ll open an issue for the Knative Eventing folks 😃
I think from this side, nothing else can be done, so I’m going to close this issue, reopen if you need it.