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

Respecting RSH8b - generate new local device's id and secret on push activation #1256

Open
maratal opened this issue Dec 22, 2021 · 4 comments
Assignees
Labels
breaking Backwards incompatible changes made to the public API.

Comments

@maratal
Copy link
Collaborator

maratal commented Dec 22, 2021

RSH8b:

The LocalDevice id and deviceSecret attributes are generated, and persisted as part of the LocalDevice state, when required by step (RSH3a2b) in the Activation State Machine.

Specs state that device's id and deviceSecret should be created on event CalledActivate. Whereas current implementation creates them lazily in the ARTRest.device property.

More details here

Related to issue

┆Issue is synchronized with this Jira Task by Unito

@lawrence-forooghian
Copy link
Collaborator

This might require a breaking change to the library (making ARTDeviceDetails.id nullable), and hence require a major version bump. I still think it's a good thing to do, though.

@maratal
Copy link
Collaborator Author

maratal commented Jan 5, 2022

This might require a breaking change to the library (making ARTDeviceDetails.id nullable), and hence require a major version bump. I still think it's a good thing to do, though.

Maybe nullable ARTDeviceDetails.id should be done here? #1259 WDYT @lawrence-forooghian

@lawrence-forooghian
Copy link
Collaborator

I think they can probably be done as separate things – we have no need to make it nullable for the solution of #1259, since in the solution #1259 there's never a moment when the device is in a "reset" state (i.e. it always has an id, secret, etc).

@maratal
Copy link
Collaborator Author

maratal commented Jan 5, 2022

I think they can probably be done as separate things – we have no need to make it nullable for the solution of #1259, since in the solution #1259 there's never a moment when the device is in a "reset" state (i.e. it always has an id, secret, etc).

As I've stated some time ago here #1184 (comment) it's actually in a reset state until app relaunched.

@AndyNicks AndyNicks added the push label May 3, 2022
@QuintinWillison QuintinWillison added bug Something isn't working. It's clear that this does need to be fixed. breaking Backwards incompatible changes made to the public API. and removed push labels Dec 6, 2022
@sync-by-unito sync-by-unito bot removed the bug Something isn't working. It's clear that this does need to be fixed. label Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Backwards incompatible changes made to the public API.
Development

No branches or pull requests

4 participants