[regression] Webbrowser tries to open file:///usr/share/webbrowser-app/'URL' instead of the URL directly

[Ursula Junque]

I notice this one while using Facebook webapp. It starts working fine, when I click on a link it opens the webbrowser as it should, but after a while when I click the link it goes to the webbbrowser window and fails trying to open "file:///usr/share/webbrowser-app/'URL'" instead.

Steps to reproduce:
1) adb shell; su - phablet;
2) upstart-app-launch webbrowser-app http://www.ubuntu.com
Browser should open trying to load www.ubuntu.com.
3) Run the same command again.
Browser will open displaying error message:
"Network Error
It appears you are having trouble viewing: file:///usr/share/webbrowser-app/'http://www.ubuntu.com'.
Ubuntu suggests you check your network settings and try refreshing the page."

Currently using r27 with channel devel-proposed, read-only, in a Nexus 4. This problem wasn't happening with only devel (r10).

[Alan Pope 🍺🐧🐱 🦄 (popey)]

Confirmed it happens on image 27 but not image 10.

[Olivier Tilloy (osomon)]

It appears that the URLs are now being passed to the application with surrounding single quotes, i.e. "'http://ubuntu.com'" instead of "http://ubuntu.com". I’ll dig to understand whether that’s a regression in the URL handler service, or if the application is supposed to trim those extra single quotes.

[Ted Gould (ted)]

Seems this the case of a single URL, we don't run it through the shell unescaping. Tested all the hard cases :-)

[Ricardo Salveti (rsalveti)]

Critical as it's a regression.

[PS Jenkins bot (ps-jenkins)]

Fix committed into lp:upstart-app-launch at revision 86, scheduled for release in upstart-app-launch, milestone Unknown

[Launchpad Janitor (janitor)]

This bug was fixed in the package upstart-app-launch - 0.3+14.04.20131126-0ubuntu1

---------------
upstart-app-launch (0.3+14.04.20131126-0ubuntu1) trusty; urgency=low

  [ Ted Gould ]
  * Bumping to 0.3 to reflect that we added a new env var to the library
    which is needed for testing URL Dispatcher
  * Reenable ZG and port to libzg-2.0. (LP: #1197569)
  * Ensure quoted single URIs passed to a %U are unquoted. (LP:
    #1253703)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 86
 -- Ubuntu daily release <email address hidden> Tue, 26 Nov 2013 02:07:13 +0000

[Ursula Junque (ursinha)]

I'm using r34 and the bug is still here. I am able to reproduce with the same steps in bug description.

[Oliver Grawert (ogra)]

Works fine for me on r31 if i put the URL in quotes ...

[Didier Roche-Tolomelli (didrocks)]

I think we need someone from the upstart-app-launch team to look at it

[Launchpad Janitor (janitor)]

Status changed to 'Confirmed' because the bug affects multiple users.

[Tiago Salem Herrmann (tiagosh)]

We are having similar problems on messaging-app and dialer-app receiving quoted urls and consequently QUrl is unable to parse them.
I suppose the code that is adding the quotes is this:
http://bazaar.launchpad.net/~indicator-applet-developers/upstart-app-launch/trunk.14.04/view/head:/libupstart-app-launch/upstart-app-launch.c#L96

[Ted Gould (ted)]

Could you add what the apps are getting as parameters and what their Exec line is. Thanks!

[Ted Gould (ted)]

Ah, I see now. Fixed a different bug. So this is a second activation bug.

[Tiago Salem Herrmann (tiagosh)]

Reading from command line is fine, the problem only happens when apps receive url's from the UriHandler qml component.
Those are quoted.

the exec line is: messaging-app message:///12345

[Charles Kerr (charlesk)]

Confirmed that lp:~ted/upstart-app-launch/second-exec-quotes works for me on the phone:

running "upstart-app-launch webbrowser-app http://www.ubuntu.com" twice from the command line as per the problem statement now works.

[PS Jenkins bot (ps-jenkins)]

Fix committed into lp:upstart-app-launch at revision 88, scheduled for release in upstart-app-launch, milestone Unknown

[Launchpad Janitor (janitor)]

This bug was fixed in the package upstart-app-launch - 0.3+14.04.20131209-0ubuntu2

---------------
upstart-app-launch (0.3+14.04.20131209-0ubuntu2) trusty; urgency=low

  * Restrict on which arch we build url-dispatcher due to the ust build-dep
 -- Didier Roche <email address hidden> Mon, 09 Dec 2013 13:47:51 +0100