Re: [webrtc-pc] Validate an ICE server url is missing length check for username (#3049)

I was hoping to see somewhere in https://w3c.github.io/webrtc-pc/#dfn-validate-an-ice-server-url a sentence saying "if the URL doesn't conform to the syntax in RFC 8489, throw (something)", but there is no such generic call-out.
In general, I prefer to incorporate specific limits by reference, not by value ("check against 509"), in case the reference gets updated - but I agree that consistent behavior is nice, so we should specify it.

(I believe the "don't throw just ignore" was done because it was easier to do at the level in the stack where I added the check, but checking earlier is fine!)

-- 
GitHub Notification of comment by alvestrand
Please view or discuss this issue at https://github.com/w3c/webrtc-pc/issues/3049#issuecomment-2850726447 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Monday, 5 May 2025 11:41:35 UTC