-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
WHEP - ETag header is E-Tag #2396
Comments
aler9
added a commit
that referenced
this issue
Sep 23, 2023
aler9
added a commit
that referenced
this issue
Sep 23, 2023
Fixed in v1.1.1 |
Note that this is a breaking change - players expecting an |
This issue is being locked automatically because it has been closed for more than 6 months. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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
The text was updated successfully, but these errors were encountered: