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

Server Sync Login Fails #32

Open
baarkerlounger opened this issue Jan 19, 2021 · 6 comments
Open

Server Sync Login Fails #32

baarkerlounger opened this issue Jan 19, 2021 · 6 comments

Comments

@baarkerlounger
Copy link

baarkerlounger commented Jan 19, 2021

One the first greeter page I try to set up sync with my Standard Notes account. The synchronization drop down gives me the option https://sync.standardnotes.org so I pick that and click login expecting to be prompted for credentials.

Instead I get: Login failed: error decoding response body: missing field 'pw_cost' at line 1 column 140

Trying to run this on Mobian (Debian) on a PinePhone. I installed by running cargo build --release, cargo run --release --bin iridium

This looks like it may be due to changes in protocol version. https://github.com/standardnotes/snjs/blob/master/specification.md

https://github.com/standardnotes/syncing-server/blob/develop/spec/controllers/api/auth_controller_spec.rb#L491

@matze
Copy link
Owner

matze commented Jan 19, 2021

Ugh, I run a standalone sync server so I missed yet another update to the protocol 🙄

Hopefully

The 004 protocol upgrade centers around a system that makes it easy and painless to upgrade to a future protocol version, as well as more modern cryptographic primitives.

will be true this time. I will keep you updated.

@baarkerlounger
Copy link
Author

Not sure if this is helpful at all but seemed similar.

tannercollin/standardnotes-fs#32

@matze
Copy link
Owner

matze commented Feb 2, 2021

I started adding a few things here and there and added facilities to better test the login procedure. But yes, it still takes a bit of time to add proper support for 004. Thanks for the hint though, I will monitor that ticket.

@baarkerlounger
Copy link
Author

There's now also a reference implementation in Python for v004.

https://gist.github.com/tannercollin/d7474c6afba0dc2026dd996b9dedf197

@baarkerlounger
Copy link
Author

Is this still actively WIP? Was/is there a branch or anything that could be looked at or tested?

@matze
Copy link
Owner

matze commented Dec 29, 2021

I am on vacation and do not have a clone with me. But I do remember working on it in a local branch but at some point stalled :-|

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

2 participants