Skip to content

client: change mapping between URL and tcURL, app, streamKey - #94

Merged
aler9 merged 1 commit into
mainfrom
fix/client-url
Aug 1, 2026
Merged

aler9 merged 1 commit into
mainfrom
fix/client-url

Conversation

@aler9

@aler9 aler9 commented Aug 1, 2026 •

Copy link
Copy Markdown
Member

Fixes: bluenviron/mediamtx#4676

URLs passed to clients are now mapped into RTMP-native fields (tcURL,
app, streamKey) in this way: tcURL contains URL without credentials and
without fragment, app contains path and query of tcURL, streamKey
contains the fragment.

@codecov

codecov Bot commented Aug 1, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.03%. Comparing base (cf5af07) to head (774f6d9).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
+ Coverage   71.99%   72.03%   +0.03%     
==========================================
  Files          64       64              
  Lines        4014     3998      -16     
==========================================
- Hits         2890     2880      -10     
+ Misses        834      828       -6     
  Partials      290      290              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

(bluenviron/mediamtx#4676)

URLs passed to clients are now mapped into RTMP-native fields (tcURL,
app, streamKey) in this way: tcURL contains URL without credentials and
without fragment, app contains path and query of tcURL, streamKey
contains the fragment.
@aler9
aler9 merged commit f426009 into main Aug 1, 2026
10 checks passed
@aler9
aler9 deleted the fix/client-url branch August 1, 2026 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[rtmp] RTMP client URI processing violates URI semantics, is not interoperable

1 participant