CasperVerswijvelt / Better-Internet-Tiles

Bring back Wi-Fi and mobile data tiles on Android 12 or higher + a better unified internet tile
GNU General Public License v3.0
166 stars 13 forks source link

Hide SSID from WiFi tile #30

Closed oSumAtrIX closed 1 month ago

oSumAtrIX commented 11 months ago

Issue

On Google Pixel devices you have to enable or disable recording from the notification center. This exposes the network SSID in the recording. The SSID can actually be used to identify ones location thanks to city network scanning, meaning, if you share the recording online, one can lookup your address.

Solution

Add an option to hide the network SSID. It could say "Internet" or "Connected" for example.

CasperVerswijvelt commented 2 months ago

Discovered that android 15 added support for detecting when screen capture is active: https://alisubhani313.medium.com/android-15s-screen-recording-detection-27349f8873cf

We could add a setting with multiple options: show, hide, hide during recording

Thoughts?

oSumAtrIX commented 2 months ago

Why should show and hide be an option? The only relevant toggle is the last.

CasperVerswijvelt commented 2 months ago

Why should show and hide be an option? The only relevant toggle is the last.

Recording detection is only available on android 15, so earlier versions wouldn't have that option

oSumAtrIX commented 2 months ago

I guess you would show the show/hide toggle on A14 and the hide during recording on A15? This would work but may put off people who were on A14 and updated to A15. So perhaps it really is the best option to show all three.

CasperVerswijvelt commented 2 months ago

the hide during recording setting is unfortunately not going to work, as the app only detects it when the app is in the foreground... I don't get the callback when screen recording and the quick settings are opened

oSumAtrIX commented 2 months ago

That's okay. Hiding works fine. Another suggestion for hiding the SSID would be to simply display "Hidden" as the SSID when you are connected

CasperVerswijvelt commented 1 month ago

This is implemented and will be in the next release

image