-
Notifications
You must be signed in to change notification settings - Fork 259
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
feat: add support for protocol version 8 #1905
base: main
Are you sure you want to change the base?
Conversation
07f3803
to
7dca883
Compare
Add support for the planned changes in protocol version 8. See: https://invent.kde.org/network/kdeconnect-android/-/commit/454b2fd5d7b3162443fbec4a6f0b7cafa1b554ee
7dca883
to
3af22bb
Compare
@andyholmes Yesterday Albert also updated the protocol docs to make it required that devices have UUID-format device IDs, the definition is now:
I feel like I still have some devices on my pairing list that use a shorter ID string. And even if not, GSConnect IDs are currently real UUIDs (with hyphens instead of underscores). I wonder if we need code to migrate devices to new-format IDs? |
I think the shorter codes are old Android hostnames, but yeah, the certificate & device ID generation needs to be updated.
I don't think true migration will be possible, since the device ID is directly tied to the certificate. It's just unavoidable that folks will have to repair some devices I suppose detecting a certificate with an old ID is possible and then showing a popup informing the user. |
Oh, yeah, but mostly I meant that there are a lot of (GSConnect) installations out there with And then they'll also have to re-pair all of their devices. (Edit: Sucks that their app preferences will all get effectively wiped out, since they'll have to reconfigure everything on the Android (or other-remote) end for the new pairing. On the local end, since it'll be the same remote device ID I believe the preferences should be carried over.) |
Are these problems specific to gsconnect? Has the KDE side been using fake_uuids_with_underscores already? |
Apparently yes, Albert's comment on making the change was:
However, while it's true GSConnect has been using UUIDs as device IDs, we've been using them with hyphens instead of underscores, while KDE Connect has apparently been doing the reverse. |
I opened #1910 to track the device ID format issue. |
Add support for the planned changes in protocol version 8.
See: https://invent.kde.org/network/kdeconnect-android/-/commit/454b2fd5d7b3162443fbec4a6f0b7cafa1b554ee