bluenviron / mediamtx

Ready-to-use SRT / WebRTC / RTSP / RTMP / LL-HLS media server and media proxy that allows to read, publish, proxy, record and playback video and audio streams.
MIT License
11.4k stars 1.45k forks source link

WHEP - ETag header is E-Tag #2396

Closed mfchiz closed 11 months ago

mfchiz commented 11 months ago

I believe the header "E-Tag" is being used for the the WHEP handshake instead of "ETag" per the spec for WHEP.

https://www.ietf.org/id/draft-murillo-whep-02.html

Which version are you using?

v1.1.0

Which operating system are you using?

Describe the issue

I created a Unity api to view WHEP sources. The code failed looking for "ETag" and it works when "E-Tag" is used. I believe the standard is "ETag"

Describe how to replicate the issue

Request a WHEP url and review the server logs and see what http headers are being returned.

Did you attach the server logs?

no

Did you attach a network dump?

no

aler9 commented 11 months ago

Fixed in v1.1.1

jwalton commented 11 months ago

Note that this is a breaking change - players expecting an E-Tag will fail to find one.

github-actions[bot] commented 5 months ago

This issue is being locked automatically because it has been closed for more than 6 months. Please open a new issue in case you encounter a similar problem.