Archived
Solve issues related to WebRTC #1773
Merged
Zenithium
merged 7 commits from 2020-03-10 16:15:15 +00:00
patch-4 into master
Dismiss Review
Are you sure you want to dismiss this review?
Labels
Clear labels
:mag:🤖 Search Engines
approved
dependencies
duplicate
feedback wanted
high priority
I2P
iOS
low priority
OS
Self-contained networks
Social media
stale
streaming
todo
Tor
WIP
wontfix
XMPP
[m]
₿ cryptocurrency
ℹ️ help wanted
↔️ file sharing
⚙️ web extensions
✨ enhancement
❌ software removal
💬 discussion
🤖 Android
🐛 bug
💢 conflicting
📝 correction
🆘 critical
📧 email
🔒 file encryption
📁 file storage
🦊 Firefox
💻 hardware
🌐 hosting
🏠 housekeeping
🔐 password managers
🧰 productivity tools
🔎 research required
🌐 Social News Aggregators
🆕 software suggestion
👥 team chat
🔒 VPN
🌐 website issue
🚫 Windows
👁️ browsers
🖊️ digital notebooks
🗄️ DNS
🗨️ instant messaging (im)
🇦🇶 translations
approved, waiting for a PR
Pull requests that update a dependency file
The Invisible Internet Project (I2P)
Operating Systems
A label for stalebot if it gets added
Anything related to media streaming.
Anything covering the Tor network
active work in progress, do not merge or PR (yet)!
Issues or bugs that will not be fixed and/or do not have significant impact on the project.
Extensible Messaging and Presence Protocol
Matrix protocol
Browser Extension related issues
Correction of content on the website
Firefox & forks, about:config etc.
Anything primarily related to site cleanup.
Virtual Private Network
*Technical* issues with the website.
Domain Name System
Anything covering a translated version of the site
No labels
Milestone
No items
No Milestone
No due date set.
Dependencies
No dependencies set.
Reference: privacyguides/privacytools.io#1773
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Description
Closes: #1483
Closes: #298
Closes: #1522
Deploy preview for privacytools-io ready!
Built with commit
4d8004e969https://deploy-preview-1773--privacytools-io.netlify.com
There seems to be conflicting information for the interface for doing this. @JonahAragon can you check that these steps are correct? Don't know anyone else who's an Apple user lol
The menu option you've mentioned here (Remove Legacy WebRTC API) does not exist in Safari. There is a menu option "Enable Legacy WebRTC API" in the WebRTC menu:
...but it appears to be disabled by default in Catalina. I've run some tests and I can't see any evidence that WebRTC implemented in Safari leaks your IP address. I would change your text to:
Also, I'm not sure if this PR actually closes #298?Seems to in your latest commit 👍Thanks for the assist, they don't call you iJonah for nothing :) The problem was that most sources for this WebRTC implementation were from 2018-2019, and posts from 2020 were just reposting the old stuff -_-
Is it worth mentioning that you don't need to disable WebRTC entirely in Safari to prevent IP leaks (and I am not sure if it is possible to disable WebRTC entirely??) -- Meaning that in the default config your IP is not leaked, but you are still able to make video calls (AFAIK? can double-check later) etc., which is definitely handy.
[citation needed]? This may be the same as Safari now (in that WebRTC is supported, because I believe it is supported on Safari on iOS) -- Alternatively if Chrome on iOS doesn't implement WebRTC, we should check if Firefox on iOS and Safari on iOS does implement it in the first place, because they would be worth adding to this list if not. I'll double-check later...
IIRC I saw the same info somewhere...
LGTM after reading the resolved comments from Jonah
Looks great so far, you have been doing lots of great things lately @zenithium , keep it up :)!