application doesn't sync, it needs to be updated.

Bug #1650046 reported by ayito
22
This bug affects 3 people
Affects Status Importance Assigned to Milestone
galileo (Ubuntu)
Fix Released
Medium
Unassigned
Yakkety
Fix Released
Medium
Dylan Aïssi

Bug Description

[Impact]
The galileo package is unusable in Yakkety. It can not be able to connect to fitbit server.
The version 0.5.0 of galileo target a wrong fitbit server.

[Test Case]
Try to synchronize a fitbit device will produce the bug.

[Regression Potential]
No potential regression since the patch only change the URL of the fitbit server.
The "new" url of the fitbit server is the same as in galileo < 0.5.0 and galileo > 0.5.0.
The wrong fitbit server was only set in galileo 0.5.0.
=============================================

I did report the bug to the developer, you can read it here:

https://bitbucket.org/benallard/galileo/issues/311/a-serious-error-happened-which-is-probably

Now I copy the information of the link above, here:

------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------
$ sudo galileo
2016-12-14 11:24:29,259:CRITICAL: # A serious error happened, which is probably due to a
2016-12-14 11:24:29,260:CRITICAL: # programming error. Please open a new issue with the following
2016-12-14 11:24:29,260:CRITICAL: # information on the galileo bug tracker:
2016-12-14 11:24:29,260:CRITICAL: # https://bitbucket.org/benallard/galileo/issues/new
2016-12-14 11:24:29,262:CRITICAL: # /usr/bin/galileo: 0.5
# Python: 3.5.2+ (default, Sep 22 2016, 12:18:14) [GCC 6.2.0 20160927]
# Platform: Linux willy 4.8.0-31-generic #33-Ubuntu SMP Wed Dec 7 15:14:02 UTC 2016 x86_64 x86_64
# pyusb: 1.0.0
# requests: 2.10.0
# yaml: 3.11 (with libyaml)
2016-12-14 11:24:29,263:CRITICAL: # Last communications:
2016-12-14 11:24:29,263:CRITICAL: # < 20 01 54 65 72 6D 69 6E 61 74 65 4C 69 6E 6B 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
2016-12-14 11:24:29,263:CRITICAL: # <
2016-12-14 11:24:29,263:CRITICAL: # > 02 01
2016-12-14 11:24:29,263:CRITICAL: # < 16 08 02 05 65 9E A6 84 BE C4 74 04 00 20 00 00 FF E7 01 00 02 00 00 00 00 00 00 00 00 00 00 00
2016-12-14 11:24:29,263:CRITICAL: # > 1A 04 BA 56 89 A6 FA BF A2 BD 01 46 7D 6E 00 00 AB AD 00 FB 01 FB 02 FB A0 0F
2016-12-14 11:24:29,263:CRITICAL: # < 20 01 53 74 61 72 74 44 69 73 63 6F 76 65 72 79 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
2016-12-14 11:24:29,263:CRITICAL: # < 13 03 B4 37 13 FE 7D F5 01 D5 06 12 04 CD 92 00 00 00 FB 00 00 00 00 00 00 00 00 00 00 00 00 00
2016-12-14 11:24:29,263:CRITICAL: # < 03 02 01 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
2016-12-14 11:24:29,263:CRITICAL: # > 02 05
2016-12-14 11:24:29,263:CRITICAL: # < 20 01 43 61 6E 63 65 6C 44 69 73 63 6F 76 65 72 79 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
2016-12-14 11:24:29,263:CRITICAL: #
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/galileo/main.py", line 288, in main
    }[config.mode](config)
  File "/usr/lib/python3/dist-packages/galileo/main.py", line 193, in sync
    for tracker in syncAllTrackers(config):
  File "/usr/lib/python3/dist-packages/galileo/main.py", line 71, in syncAllTrackers
    if not galileo.requestStatus(not config.httpsOnly):
  File "/usr/lib/python3/dist-packages/galileo/net.py", line 187, in requestStatus
    self.post('status')
  File "/usr/lib/python3/dist-packages/galileo/net.py", line 134, in post
    r.raise_for_status()
  File "/usr/lib/python3/dist-packages/requests/models.py", line 844, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 401 Client Error: Unauthorized for url: https://api.fitbit.com:443/tracker/client/message

------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------

The response for the developer Benoît Allard is:

I see you are using an older version with a wrong url builtin to connect to the fitbit server. The url should be corrected to client.fitbit.com. Either use a newer version (0.5.1) or modify the url in the code.

-----

I followed his instructions and now I can sync the device.
------
Thank you very much, now it is working. This is what I did, step by step:

$ cd /usr/lib/python3/dist-packages/galileo/
$ sudo grep -rl "api.fitbit.com" ./
./main.py
./__pycache__/main.cpython-35.pyc
$ sudo vim main.py

on line 37:
    galileo = GalileoClient('https', 'api.fitbit.com',
                            'tracker/client/message')

I made the suggested change, so now it is:
    galileo = GalileoClient('https', 'client.fitbit.com',
                            'tracker/client/message')

The saved the file.
------

So, please, could you check on this and update the package to its latest version, so everyone can benefit.

Thank you very much.

Eduardo

Revision history for this message
Torsten Franz (torsten.franz) wrote :

Hi Eduardo,

thank you for reporting this issue and for your help to make Ubuntu better.

In Zesty (17.04) we have a newer version:
Version 0.5.1-4 uploaded on 2016-11-11

It should be fix in 17.04 by default.

Bests,
Torsten

Revision history for this message
Launchpad Janitor (janitor) wrote :

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

Changed in galileo (Ubuntu):
status: New → Confirmed
Revision history for this message
ayito (ayito) wrote :

Hi again,

Thanks for the advice on installing 0.5.1-4, though I haven't done it since the fix I applied solved my problem and it is just good enough for me for the time being.

Anyway, since I also reported the bug to the developer, I also got answers to it over there, so I invite you to go back there and read the newer posts. They recommend me to ask for a SRU (Stable Release Update), and will back me up in doing so, which I will be grateful if they do. Though I am a first timer doing this and have no clue how to do it.

The link seems to be this:
https://wiki.ubuntu.com/StableReleaseUpdates

and the proposed fix is this:
https://bitbucket.org/benallard/galileo/issues/attachments/311/benallard/galileo/1481921710.25/311/0001-Fix-fitbit-server-url.patch

As stated in the thread:
https://bitbucket.org/benallard/galileo/issues/311/a-serious-error-happened-which-is-probably

So please, do the stable release update for Yakkety Yak. I guess this is kind of a mess of bug report, not really well formatted, but I think you understood the problem and also can check the other thread for further explanantions. Me being a first timer in bug reporting, next time I'll try to adhere to the explanations mentioned in Stable Release Updates link.

Thanks again!

Revision history for this message
Launchpad Janitor (janitor) wrote :

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

Dylan Aïssi (daissi)
affects: hep-galileo → ubuntu
Changed in ubuntu:
status: New → Confirmed
Revision history for this message
Dylan Aïssi (daissi) wrote :

Hi,
I am the Debian maintainer of galileo. This bug is fixed upstream and in Zesty, so it should target Yakkety since it make the package unusable in Yakkety only. Moreover, it is fixed by a very simple (attached) patch.

Best regards.

Changed in galileo (Ubuntu):
status: Confirmed → Fix Released
Revision history for this message
Dylan Aïssi (daissi) wrote :
Revision history for this message
C de-Avillez (hggdh2) wrote :

reverting to Triaged, fix not yet applied

Changed in galileo (Ubuntu):
status: Fix Released → Triaged
Revision history for this message
C de-Avillez (hggdh2) wrote :

Oops. It is fixed on Zesty, so it will need SRUs.

Changed in galileo (Ubuntu):
status: Triaged → Fix Released
C de-Avillez (hggdh2)
Changed in galileo (Ubuntu Yakkety):
status: New → Triaged
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "0001-Fix-fitbit-server-url.patch" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags: added: patch
Revision history for this message
Robie Basak (racb) wrote :

Dylan's patch looks fine. Thank you for preparing the fix! But please could someone follow https://wiki.ubuntu.com/StableReleaseUpdates#Procedure to get this fixed in Yakkety? Step 3 has not been done, and this will block acceptance into Yakkety.

Anybody affected can do this. No need to be a developer. Just read the "Why" and "When" sections above "Procedure" for the background. Thanks!

Dylan Aïssi (daissi)
description: updated
Mathew Hodson (mhodson)
no longer affects: Ubuntu Yakkety
no longer affects: ubuntu
Changed in galileo (Ubuntu):
importance: Undecided → Medium
Changed in galileo (Ubuntu Yakkety):
importance: Undecided → Medium
tags: added: bitesize
Revision history for this message
Dylan Aïssi (daissi) wrote :

Hi,
Any news for this simple SRU? :-)

Revision history for this message
Mattia Rizzolo (mapreri) wrote :

Uploaded, thanks for your contribution :)

Changed in galileo (Ubuntu Yakkety):
assignee: nobody → Dylan A. (dylan.a)
status: Triaged → In Progress
Revision history for this message
Sebastien Bacher (seb128) wrote :

The upload is in the SRU queue waiting for review so unsubscribing sponsors

Revision history for this message
Dylan Aïssi (daissi) wrote :

Hi,
This SRU is in the upload queue and waiting to be approved since two weeks.
I know that galileo is not the most priority :) but can we do something to accelerate the process here?

Best,
Dylan

Revision history for this message
Robie Basak (racb) wrote :

Sorry for the delay. We had quite a backlog build up over the Christmas period, when most of the SRU team members were on vacation.

@mapreri, please note that I had to re-upload to fix a couple of things:

1) The changes file didn't include Launchpad-Bugs-Fixed:, which breaks SRU process tooling. Ubuntu's tooling (devscripts -> dpkg-genchanges?) does this automatically. If you use a Debian system to sponsor an upload, you have to patch or add this manually.

2) I ran "update-maintainer" to implement https://wiki.ubuntu.com/DebianMaintainerField

Changed in galileo (Ubuntu Yakkety):
status: In Progress → Fix Committed
tags: added: verification-needed
Revision history for this message
Robie Basak (racb) wrote : Please test proposed package

Hello ayito, or anyone else affected,

Accepted galileo into yakkety-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/galileo/0.5.0-1ubuntu1 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 on 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/PerformingSRUVerification . Thank you in advance!

Revision history for this message
Dylan Aïssi (daissi) wrote :

Hi,
I have tested 0.5.0-1ubuntu1 with my own fitbit and the sync works fine!

tags: added: verification-done
removed: verification-needed
Revision history for this message
ayito (ayito) wrote :

Hi,
I can confirm also that it works with Fitbit Charge HR, it connects to the server.
Also I have checked the file "main.py" and the GalileoClient has the parameter on 'client.fitbit.com' as it should.

So I think it's ok now.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package galileo - 0.5.0-1ubuntu1

---------------
galileo (0.5.0-1ubuntu1) yakkety; urgency=medium

  * Backport upstream commit 40abc63 to fix the URL of Fitbit server. (LP: #1650046)

 -- Dylan Aïssi <email address hidden> Fri, 16 Dec 2016 21:37:39 +0100

Changed in galileo (Ubuntu Yakkety):
status: Fix Committed → Fix Released
Revision history for this message
Robie Basak (racb) wrote : Update Released

The verification of the Stable Release Update for galileo has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.