Skip to content

0.30.0: WHEP credentials in the URL as Basic authorization - #81

Merged
mhoogenbosch merged 2 commits into
masterfrom
whep-basic-auth
Oct 31, 2026
Merged

mhoogenbosch merged 2 commits into
masterfrom
whep-basic-auth

Conversation

@mhoogenbosch

@mhoogenbosch mhoogenbosch commented Oct 31, 2026 •

Copy link
Copy Markdown
Owner

A WHEP endpoint behind basic authentication, such as go2rtc with api.username and api.password, did not work: the URL carried the credentials, which fetch() refuses, and the player page got them in its base URL as well.

  • splitUrlCredentials() splits user:password@ off the URL (percent-decoded), as a plain function with unit tests
  • the WHEP page and the camera wall (grid) send them as an Authorization: Basic header; the page origin no longer contains them
  • a URL without credentials behaves as before
  • version 0.30.0 (versionCode 69), readme and changelog

Tests: testDebugUnitTest 43/43 (5 new), assembleDebug OK. The page's Base64 function was checked against Python, including UTF-8.

🤖 Generated with Claude Code

mhoogenbosch and others added 2 commits October 31, 2026 09:30
A WHEP endpoint behind basic authentication (go2rtc with api.username and
api.password) failed: the URL carried the credentials, which fetch() refuses,
and the page's base URL got them too. The credentials are now split off the
URL in Kotlin and sent by the player page as an Authorization: Basic header,
for a single stream and for the tiles of a camera wall. A URL without
credentials behaves as before.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mhoogenbosch mhoogenbosch changed the title WHEP: credentials in the URL as Basic authorization 0.30.0: WHEP credentials in the URL as Basic authorization Oct 31, 2026
@mhoogenbosch
mhoogenbosch merged commit 5b9341b into master Oct 31, 2026
2 checks passed
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.

1 participant