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

Update device capabilities #353

Merged
merged 1 commit into from
Jan 30, 2025
Merged

Conversation

Schmiddiii
Copy link
Contributor

This fixes a 409 error received when linking devices. Fixes #352.

@gferon
Copy link
Collaborator

gferon commented Jan 28, 2025

Nice! I suppose you've tried it with either presage-cli or Flare?

@Schmiddiii
Copy link
Contributor Author

I tried it with Flare. Worked for me.

@rubdos
Copy link
Member

rubdos commented Jan 28, 2025

Can we at least document what this new capability is? Just advertising that we're capable of handling it might get us into more trouble than not being able to link.

@gferon
Copy link
Collaborator

gferon commented Jan 28, 2025

I couldn't figure it out from the source of SD 😢

@rubdos
Copy link
Member

rubdos commented Jan 28, 2025

Signal Server code:

  STORAGE_SERVICE_RECORD_KEY_ROTATION("ssre2", AccountCapabilityMode.ALL_DEVICES, true, true);

Signal Android:

@JsonProperty("ssre2") val storageServiceEncryptionV2: Boolean

I assume it pertains to backups.

This fixes a 409 error received when linking devices.
Fixes whisperfish#352.
@Schmiddiii
Copy link
Contributor Author

I added a little comment to the field, referencing both the Signal Android and Signal Server source code you mentioned.

@Schmiddiii
Copy link
Contributor Author

Can this be merged or is there anything else I should do in this PR?

@gferon gferon merged commit 43aa16f into whisperfish:main Jan 30, 2025
5 of 6 checks passed
@Schmiddiii Schmiddiii deleted the fix-409-linking branch January 30, 2025 09:13
Schmiddiii added a commit to Schmiddiii/presage that referenced this pull request Jan 30, 2025
Contains an important fix for linking devices

Fixes whisperfish/libsignal-service-rs#352
via upstream PR whisperfish/libsignal-service-rs#353.
gferon pushed a commit to whisperfish/presage that referenced this pull request Jan 30, 2025
Contains an important fix for linking devices

Fixes whisperfish/libsignal-service-rs#352
via upstream PR whisperfish/libsignal-service-rs#353.
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

Successfully merging this pull request may close these issues.

409 error when linking without any other secondary devices existing
3 participants