1797822 - Site permission prompts show the full origin like https://permission.site:443 instead of just the hostname like permission.site

Status: NEW

(Fenix :: Browser Engine, enhancement, P3)

Description

[Chris Peterson [:cpeterson]]

Steps to reproduce

1. Load https://permission.site/
2. Tap the "Camera" button.
3. You will see a permission prompt: "Allow https://permission.site:443 to use your camera and microphone?"

Actual result

Fenix's permission prompt shows: Allow https://permission.site:443 to use your camera and microphone?.

Expected result

Desktop's permission prompt shows: Allow permission.site to use your camera and microphone? without the https:// scheme or :443 port. Fenix should use desktop's shorter string, especially since Fenix has less space to show the string.

Comment 1

[Junaidjd1014]

i have a similar issue while i using firfox browser my dowloadhub movie site https://downloadhubmovie.com/ not working properly and Allow permission always whenever i refresh it.

Comment 2

[Chris Peterson [:cpeterson]]

Enhancements should have severity N/A.

Comment 3

[Chris Peterson [:cpeterson]]

Attachment: permission_prompt_origin.png

Screenshot of a permission prompt with origin

Comment 4

[Chris Peterson [:cpeterson]]

Attachment: Bug 1797822 - Only show URL scheme and port in site permission prompts for protocols other than HTTPS. r?tthibaud

Firefox Android's site permission prompts currently show the site's origin (which incudes the URL's protocol scheme and port number) like "Allow https://permission.site:443 to use your camera and microphone?". With this change, Firefox Android will only show the host, like Firefox desktop does: "Allow permission.site to use your camera and microphone?".

Firefox Android's site permission prompts used to show just the host, but switched to origin in this 2021 commit:

https://github.com/mozilla/gecko-dev/commit/497d1d2c35aa551a8c35c115d1fe6e4f1d4756b7

Comment 5

[Chris Peterson [:cpeterson]]

Attachment: Bug 1797822 - Rename some SitePermissions variables from host to origin. r?tthibaud

Since Gecko scopes permissions to origins (like "https://www.example.com:443"), we want all permission checks to use origins, not just hostnames like "www.example.com". Some SitePermissions code was incorrectly naming some variables "host" when they actually stored an origin.