[webapp-container] short url patterns are considered invalid
Bug #1358399 reported by
David Barth
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
webbrowser-app |
Fix Released
|
Medium
|
David Barth | ||
webbrowser-app (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
URL patterns without a subdomain are considered invalid.
Testcase: http?:/
Related branches
lp:~dbarth/webbrowser-app/short-url-patterns
- PS Jenkins bot: Approve (continuous-integration)
- Alexandre Abreu (community): Approve
- Ubuntu Phablet Team: Pending requested
-
Diff: 49 lines (+22/-1)2 files modifiedsrc/app/webcontainer/url-pattern-utils.cpp (+14/-1)
tests/unittests/container-url-patterns/tst_ContainerUrlPatternsTests.cpp (+8/-0)
Changed in webbrowser-app: | |
importance: | Undecided → Medium |
assignee: | nobody → David Barth (dbarth) |
status: | New → Confirmed |
Changed in webbrowser-app: | |
status: | Confirmed → Fix Committed |
Changed in webbrowser-app: | |
status: | Fix Committed → In Progress |
tags: | added: touch-2014-09-11 |
Changed in webbrowser-app: | |
status: | In Progress → Fix Committed |
Changed in webbrowser-app: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
E.g.
Install untappd app from the store.
Open untapped app
Click "sign in" link.
Note that the url to sign in is https:/ /untappd. com/login which should be covered by the pattern I am using "--webappUrlPat terns=https? ://*.untappd. com/*,https? ://untappd. com/* " but isn't. It breaks out into a new web browser instance.
Attached click for testing.