Eyevinn / srt-whep

SRT to WHEP (WebRTC)
Apache License 2.0
69 stars 9 forks source link

Invalid SDP: Client initialization not supported. #22

Closed smanemann closed 1 year ago

smanemann commented 1 year ago

Hey Again,

there seems to be another Issue, apart from the gstreamer dep., which is showing up after receiving http offers from the client:

{"v":0,"name":"srt_whep","msg":"[HTTP REQUEST - EVENT] Error encountered while processing the incoming HTTP request: Failed to process request: Invalid SDP: Client initialization not supported.\n\n","level":40,"hostname":"xxxxxxx","pid":29811,"time":"2023-06-29T12:57:38.115737688Z","target":"tracing_actix_web::middleware","line":256,"file":"/home/admin/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tracing-actix-web-0.7.5/src/middleware.rs","http.client_ip":"45.xxxx","http.scheme":"http","request_id":"1a8dc9d7-3d02-48e4-873b-197e37916d32","otel.kind":"server","otel.name":"HTTP POST /channel","http.route":"/channel","exception.details":"Failed to process request: Invalid SDP: Client initialization not supported.\n\n","http.target":"/channel","http.flavor":"1.1","http.host":"3.78.220.112:8000","http.method":"POST","http.user_agent":"Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/114.0.0.0 Safari/537.36","exception.message":"Failed to process request: Invalid SDP: Client initialization not supported.","http.status_code":400,"otel.status_code":"OK"}

I was trying to find some documentation around the tracing-actix-web middleware , but couldn't find any trace of issues with client initialization.

any idea? :)

birme commented 1 year ago

We don't support the client side init mode of WHEP. This is under discussion but as the server knows what streams it has we believe the server should provide the SDP offer

birme commented 1 year ago

Our WHEP player supports both modes at the moment

Wkkkkk commented 1 year ago

Hi @smanemann, here is a short summary of the WHEP protocol and what supports we have for now.

smanemann commented 1 year ago

Thanks for the clarfications. Is there maybe a specific adapter for your WHEP player? I was using that with the WHEP Adapter for testing when I got the error.

birme commented 1 year ago

That should work out of the box. It tries with one mode first and if that fails tries the next one

birme commented 1 year ago

Hi @smanemann, here is a short summary of the WHEP protocol and what supports we have for now.

Should we perhaps put this in a docs/whep.md in the repo @Wkkkkk ?

Wkkkkk commented 1 year ago

Related discussions are under docs/whep.md now.