URL Dispatcher Should Prompt on Bad URL
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | Canonical System Image |
Medium
|
Bill Filler | ||
| | url-dispatcher (Ubuntu) |
High
|
Ted Gould | ||
Bug Description
The Ubuntu Error Tracker has been receiving reports about a problem regarding gallery-app. This problem was most recently seen with version 13.10.0+
<https:/
Related branches
- Charles Kerr (community): Approve on 2016-10-26
- unity-api-1-bot: Approve (continuous-integration) on 2016-09-30
-
Diff: 740 lines (+352/-72)24 files modifiedCMakeLists.txt (+1/-0)
data/CMakeLists.txt (+10/-0)
data/bad-url.qml (+42/-0)
debian/apparmor/url-dispatcher-bad-url-helper (+15/-0)
debian/control (+4/-0)
debian/rules (+1/-0)
debian/url-dispatcher.install (+3/-0)
po/CMakeLists.txt (+39/-0)
po/POTFILES.in (+1/-0)
po/genpotfiles.sh (+6/-0)
service/CMakeLists.txt (+5/-12)
service/bad-url/CMakeLists.txt (+14/-0)
service/bad-url/exec-tool.c (+42/-0)
service/dispatcher.c (+4/-0)
service/overlay-tracker-iface.h (+1/-0)
service/overlay-tracker-mir.cpp (+69/-39)
service/overlay-tracker-mir.h (+22/-8)
service/overlay-tracker.cpp (+9/-0)
service/overlay-tracker.h (+1/-0)
service/url-overlay/CMakeLists.txt (+14/-0)
tests/dispatcher-test.cc (+1/-0)
tests/overlay-tracker-mock.h (+6/-0)
tests/overlay-tracker-test.cpp (+41/-12)
tests/test-config.h.in (+1/-1)
| Changed in gallery-app (Ubuntu): | |
| importance: | Undecided → High |
| Bill Filler (bfiller) wrote : | #2 |
this appears to be a webbrowser-app issue not a gallery issue, please take a look
| Changed in webbrowser-app (Ubuntu): | |
| assignee: | nobody → Olivier Tilloy (osomon) |
| importance: | Undecided → High |
| Bill Filler (bfiller) wrote : | #3 |
Seems to be 1165 occurrances of this crash according to:
https:/
| Olivier Tilloy (osomon) wrote : | #4 |
This is not a crash, it’s a recoverable problem. It seems to happen when trying to open market:// URLs in the browser (from pages that incorrectly assume that they are running on an android device).
I can easily reproduce the issue by opening a test page in the browser that contains this code:
<html>
<body>
<a href="market:
</body>
</html>
When clicking this link, I’m seeing the following error in the browser’s log:
** (process:14120): WARNING **: Unable to dispatch url 'market:
We could probably have a default application that registers as a handler for this type of URLs and informs the user that android applications are not compatible with Ubuntu (adding an ubuntu-ux task to get input from the design team).
| Changed in gallery-app (Ubuntu): | |
| status: | Confirmed → Invalid |
| Changed in webbrowser-app (Ubuntu): | |
| status: | New → Confirmed |
| Changed in ubuntu-ux: | |
| assignee: | nobody → Matthew Paul Thomas (mpt) |
| status: | New → Triaged |
| importance: | Undecided → High |
| Stuart Langridge (sil) wrote : | #5 |
In general there should probably be fallback handling in the URL dispatcher for all unknown URL schemata, I think? (Although having a dedicated handler for market: URLs may also be useful.)
| Changed in ubuntu-ux: | |
| status: | Triaged → In Progress |
| Matthew Paul Thomas (mpt) wrote : | #6 |
Specification updated. <https:/
| Changed in ubuntu-ux: | |
| status: | In Progress → Fix Committed |
| description: | updated |
| affects: | webbrowser-app (Ubuntu) → url-dispatcher (Ubuntu) |
| Changed in url-dispatcher (Ubuntu): | |
| assignee: | Olivier Tilloy (osomon) → nobody |
| summary: |
- url-dispatcher-bad-url + URL Dispatcher Should Prompt on Bad URL |
| Bill Filler (bfiller) wrote : | #7 |
This is needed for https:/
| Changed in canonical-devices-system-image: | |
| importance: | Undecided → High |
| no longer affects: | gallery-app (Ubuntu) |
| Changed in canonical-devices-system-image: | |
| milestone: | none → backlog |
| Pat McGowan (pat-mcgowan) wrote : | #8 |
Can we find someone to work on this?
| Changed in canonical-devices-system-image: | |
| assignee: | nobody → Bill Filler (bfiller) |
| status: | New → Confirmed |
| Changed in canonical-devices-system-image: | |
| importance: | High → Medium |
| Changed in url-dispatcher (Ubuntu): | |
| assignee: | nobody → Ted Gould (ted) |
| Launchpad Janitor (janitor) wrote : | #9 |
This bug was fixed in the package url-dispatcher - 0.1+17.
---------------
url-dispatcher (0.1+17.
[ Florian Boucault ]
* Crossbuilding Fixes
[ Larry Price ]
* App ID should allow package names with dots. (LP: #1591342) (LP: #1606498)
[ Ted Gould ]
* Check versions of appid:// urls to reduce the number of invalid appids
we send to UAL (LP: #1352656)
* Scope testing for the scope:// URL and overlays that should be over
the dash
* Protect the dash from getting NULL urls (LP: #1476257)
* Make sure URL Overlays are shutdown on exit
* Remove the wildcard that is confusing the update script (LP: #1461496)
* Not using a message that isn't set properly (LP: #1608009)
* Fixing tests to adjust for UAL changes
* Add a small GUI helper tool for testing
* Show error dialog over apps that use Bad URLs (LP: #1370008)
-- Ted Gould <email address hidden> Mon, 07 Nov 2016 16:33:15 +0000
| Changed in url-dispatcher (Ubuntu): | |
| status: | Confirmed → Fix Released |
| no longer affects: | ubuntu-ux |
Status changed to 'Confirmed' because the bug affects multiple users.