Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

desktop: Bluetooth download from DC fails in common use case #1002

Closed
1 of 4 tasks
janmulder opened this issue Dec 31, 2017 · 4 comments
Closed
1 of 4 tasks

desktop: Bluetooth download from DC fails in common use case #1002

janmulder opened this issue Dec 31, 2017 · 4 comments

Comments

@janmulder
Copy link
Collaborator

Describe the issue:

  • Bug
  • Change request
  • New feature request
  • Discussion request

Issue long description:

For some time, I downloaded from mobile, but back home, I like to download from the DC using the desktop application again. This is currently not working correctly. Simply go to the Import from DC screen, and as I always download this way, my BT button is default selected and the device mount point correctly filled with the following string OSTC3#05106 (00:80:25:4A:0F:C3). This nicely formatted string comes from commit de81eff (@bstoeger). Now click download and it fails (always). First I thought it had something to do with a almost end-of-live battery in my OSTC.

There is a workaround, and that points immediately to the reason of this bug. When going to the "remote Bluetooth device selection" screen first, selecting the DC and save it, and then back at the download screen, the download succeeds. Or just manually type in the BT address without the name padding.

Further investigation shows that all this is caused by passing the nicely formatted devname to libdc, and that obviously fails. The workaround route sets the devname to the BT address without the name, so that succeeds.

Operating system:

will fail on Linux, Apple, Windows.

Subsurface version:

latest master

Steps to reproduce:

see description

Current behavior:

see description

Expected behavior:

see description

Additional information:

commit de81eff is related. And I believe this is a high prio issue, as it is a very common use case, and an average user using BT/BLE download will not find the workaround instinctively.

Mentions:

@bstoeger

@atdotde
Copy link
Collaborator

atdotde commented Dec 31, 2017

Could this be related to 1c223b0 ? At the time I tested this on iOS and Mac but it could be breaking Linux (at least I didn't test).

@janmulder
Copy link
Collaborator Author

Could be, not sure. Will try this by reverting.

@bstoeger
Copy link
Collaborator

Give me a few minutes, I'm working on a fix.

@janmulder
Copy link
Collaborator Author

@atdotde: unrelated to 1c223b0.

bstoeger added a commit to bstoeger/subsurface that referenced this issue Dec 31, 2017
If the BT dialog hasn't been shown, the device name was taken from
the text field, which contained a formatted string. The device open
would then fail.

Fixes subsurface#1002

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
bstoeger added a commit to bstoeger/subsurface that referenced this issue Dec 31, 2017
Owing to bug subsurface#1002 invalid bluetooth device addresses of the form
"devicename (deviceaddress)" or "deviceaddress (devicename)" may
have found their way into the preferences. Recognize such names
and extract the correct address.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
@bstoeger bstoeger mentioned this issue Dec 31, 2017
7 tasks
bstoeger added a commit to bstoeger/subsurface that referenced this issue Dec 31, 2017
Owing to bug subsurface#1002 invalid bluetooth device addresses of the form
"devicename (deviceaddress)" or "deviceaddress (devicename)" may
have found their way into the preferences. Recognize such names
and extract the correct address.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
@dirkhh dirkhh closed this as completed in 3c022d8 Dec 31, 2017
dirkhh pushed a commit that referenced this issue Dec 31, 2017
Owing to bug #1002 invalid bluetooth device addresses of the form
"devicename (deviceaddress)" or "deviceaddress (devicename)" may
have found their way into the preferences. Recognize such names
and extract the correct address.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants