Replies: 1 comment
|
Hi, this is intentional. The setting only affects It does not disable TLS, downgrade HTTPS, or send plaintext traffic. It only skips certificate and hostname validation so playback and downloads can continue when third-party CDN endpoints, particularly Bilibili CDNs, have broken certificate configurations. This does not create a meaningful security threat because these connections only carry media traffic, not account credentials or local app data. Scoping it to a fixed hostname is not reliable because media CDN hosts can change. The current behavior is therefore an availability tradeoff for media traffic, not a global disabling of HTTPS security. |
Uh oh!
There was an error while loading. Please reload this page.
Hi,
While reviewing the codebase, I noticed that
MainActivitycallstrustEveryone()during startup:The implementation appears to globally replace the default
HostnameVerifierandX509TrustManager:From my understanding, this effectively disables hostname verification and certificate validation for
HttpsURLConnection, which could allow man-in-the-middle (MITM) attacks against HTTPS connections.I was wondering about the rationale behind this implementation.
I'm asking because globally disabling TLS verification is generally considered a significant security risk, so I'd like to better understand the reasoning before proposing any changes.
Thanks!
All reactions