Ubuntu Push client should ensure it only runs once per session.
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu Push Notifications |
Fix Released
|
High
|
John Lenton | ||
ubuntu-push (Ubuntu) |
Fix Released
|
High
|
John Lenton | ||
Trusty |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
Ubuntu Push client should ensure it only runs once per session.
[Impact]
* if two push clients connect with the same id, the last one to connects disconnects the previous ones
* a "live" client will attempt to reconnect
thus if for whatever reason two clients run in the same session, they will "fight" over it.
[Test Case]
You need:
* a device on stable
* two shells running as the phablet user
In one of the shells, do
stop ubuntu-push-client
/usr/lib/
In the other one, do
/usr/lib/
Watch the fight that ensues
[Regression Potential]
The current implementation quits the first client once the second one claims the dbus name. This *could* be disconnecting the "wrong" one, but as I have had trouble reproducing this in the wild it's hard to tell. In any case it's better than the fight that would ensue without this.
Changed in ubuntu-push: | |
assignee: | nobody → John Lenton (chipaca) |
importance: | Undecided → High |
status: | New → Fix Committed |
Changed in ubuntu-push (Ubuntu): | |
assignee: | nobody → John Lenton (chipaca) |
importance: | Undecided → High |
status: | New → In Progress |
description: | updated |
description: | updated |
Changed in ubuntu-push: | |
status: | Fix Committed → Fix Released |
Hello John, or anyone else affected,
Accepted ubuntu-push into trusty-proposed. The package will build now and be available at http:// launchpad. net/ubuntu/ +source/ ubuntu- push/0. 2.1+14. 04.20140423. 1-0ubuntu1 in a few hours, and then in the -proposed repository.
Please help us by testing this new package. See https:/ /wiki.ubuntu. com/Testing/ EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.
If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification- failed. In either case, details of your testing will help us make a better decision.
Further information regarding the verification process can be found at https:/ /wiki.ubuntu. com/QATeam/ PerformingSRUVe rification . Thank you in advance!