https://bugs.kde.org/show_bug.cgi?id=525625

Simon Redman <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://invent.kde.org/netw
                   |                            |ork/kdeconnect-android/-/co
                   |                            |mmit/1021ff53c6cb908156dedd
                   |                            |b7c93c62cd69630072
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from Simon Redman <[email protected]> ---
Git commit 1021ff53c6cb908156deddb7c93c62cd69630072 by Simon Redman, on behalf
of Matt C..
Committed on 16/09/2026 at 12:19.
Pushed by sredman into branch 'master'.

ContactsPlugin: send UID timestamps as numbers

A stray `.toString()` on the timestamp in `handleRequestAllUIDsTimestamps`
causes the String overload of `NetworkPacket.set()` to be selected instead of
the Long overload, so `kdeconnect.contacts.response_uids_timestamps` sends
timestamps as quoted JSON strings instead of numbers. Desktop KDE Connect
tolerates this via implicit type coercion, but strict clients like Valent
reject the packet.

Regression from 2d8b43d7 ("Migrate ContactsPlugin to Kotlin (conflict
resolution)"): the Java `reply.set(contactID.toString(), timestamp)` became
`set(contactID.toString(), timestamp.toString())` during the Kotlin port, which
switched it to the String overload.

Testing: verified by code inspection. The fix removes the `.toString()` so the
Long overload is used, and the desktop client parses this field as a number.

M  +1    -1   
src/main/java/org/kde/kdeconnect/plugins/contacts/ContactsPlugin.kt

https://invent.kde.org/network/kdeconnect-android/-/commit/1021ff53c6cb908156deddb7c93c62cd69630072

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to