access: rtp: case insensitive match configuration parameters
rfc3640 format parameters are not case dependent
Merge request reports
Activity
changed milestone to %4.0
added Component::Input: network streams label
assigned to @fcartegnie
- Resolved by Thomas Guillem
For the record, RFC3640 section 4.1:
MIME format parameters are not case dependent
added MRStatus::Acceptable label
- Resolved by François Cartegnie
AFAIK,
strncasecmp
is locale-dependent, and that's definitely not what the RFCs mean.
added MRStatus::InReview label and removed MRStatus::Acceptable label
- Resolved by Thomas Guillem
That does not seem correct.
fmtp
is codec-dependent, and typically the parameter names are given as an exact string - not case-insensitive. Given that SDP is officially UTF-8, it would be complete hell to do case-insensitive matching.
added 1 commit
- f01a7160 - access: rtp: case insensitive match configuration parameters
mentioned in merge request !6409 (merged)
added MRStatus::Acceptable label and removed MRStatus::InReview label
added MRStatus::Accepted label and removed MRStatus::Acceptable label
MR Acceptance result
This MergeRequest has been Accepted! Congratulations. MR acceptance checks details:
-
MR should be considered mergeable by Gitlab -
Last pipeline should be successful -
MergeRequest should have at least one external review and/or vote -
All threads should be resolved, and score >= 0 -
MergeRequest should have no activity (threads/votes) for (24h/24h)
-
added 182 commits
-
f01a7160...635962fd - 181 commits from branch
videolan:master
- bd1cd88d - access: rtp: case insensitive match configuration parameters
-
f01a7160...635962fd - 181 commits from branch
enabled an automatic merge when all merge checks for bd1cd88d pass