wish-wg / webrtc-http-ingest-protocol

WHIP - WebRTC HTTP ingest protocol draft
48 stars 10 forks source link

WHIP session definition #167

Closed cdh4u closed 8 months ago

cdh4u commented 9 months ago

Section 3:

This does not define what a WHIP session is, as the definition text contains the terminology it is supposed to define.

murillo128 commented 9 months ago

how about this?

WHIP session: Indicates the allocated HTTP resource by the WHIP
endpoint for an ongoing ingest session.
cdh4u commented 9 months ago

Your suggested fix would work, assuming that is the only thing we mean by "WHIP session". I.e., "WHIP session" does NOT refer to a session in the traditional meaning of session (SIP session etc). Personally, I would call the allocated resource something else than "session", but if the community has decided to use "session" I will not argue against it.

In the definition of WHIP session URL, I assume you should also change "WHIP resource" to "HTTP resource".

murillo128 commented 9 months ago

I agree, I used to call it WHIP resource instead of WHIP session but the httpdir review argued that this would be confusing with HTTP resource (as the WHIP endpoint is also an HTTP resource).

I would prefer avoid calling it WHIP session resource but I have not found a better naming for it yet.

cdh4u commented 9 months ago

Perhaps we could use "WHIP server endpoint" and "WHIP session endpoint"?