Skip to content
This repository was archived by the owner on Sep 29, 2023. It is now read-only.

Add Zoom URL protocol#1463

Closed
veged wants to merge 1 commit into
nativefier:masterfrom
veged:patch-1
Closed

Add Zoom URL protocol#1463
veged wants to merge 1 commit into
nativefier:masterfrom
veged:patch-1

Conversation

@veged

@veged veged commented Sep 21, 2022

Copy link
Copy Markdown

No description provided.

@ronjouch

Copy link
Copy Markdown
Contributor

@veged thanks! I remain puzzled about Firefox's allowlist. Can you test if zoommtg: links work in Firefox?

@veged

veged commented Sep 22, 2022

Copy link
Copy Markdown
Author

@veged thanks! I remain puzzled about Firefox's allowlist. Can you test if zoommtg: links work in Firefox?

@ronjouch it works fine, but with confirmation — that allowlist is about those URL protocols which could be followed without confirmation

ronjouch added a commit that referenced this pull request Nov 7, 2022
Also, document the current state of things.
Follow-up of #1463 .
@ronjouch

ronjouch commented Nov 7, 2022

Copy link
Copy Markdown
Contributor

it works fine, but with confirmation — that allowlist is about those URL protocols which could be followed without confirmation

@veged aaaaAAAAaaaaaaaaaaaah and NOW this Firefox list makes sense! I didn't test this feature in Firefox, I just re-used their list, and I was under the impression they just blocked 😄. No okay, this list is what doesn't have a confirmation, and other protocols go through a confirmation.

Well, thanks for the patch and thanks for enlightening! So, I pushed ba7244f that adds Zoom as a separate list, and documents the current state. With that, closing this PR.

@ronjouch ronjouch closed this Nov 7, 2022
Adam777Z pushed a commit to Adam777Z/nativefier that referenced this pull request Nov 9, 2022
Also, document the current state of things.
Follow-up of nativefier#1463 .
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants