Update development database setup to work with secure signup

Automattic / simplenote-electron

Simplenote for Web, Windows, and Linux

https://app.simplenote.com

GNU General Public License v2.0

4.76k stars 557 forks source link

Update development database setup to work with secure signup #2853

Open xiaofen9 opened 3 years ago

xiaofen9 commented 3 years ago

I built and ran multiple versions (run dev) of the simplenote electron apps but I encountered the same issue: simplenote wont let me login with status code 401. The devtool shows POST https://auth.simperium.com/1/history-analyst-dad/authorize/ 401

App versions that DOES not work

2.9.0 (self-built)
2.9.0 official linux built )
1.12.1 (self-built)
web app hosting on 4000

App versions that work:

2.9.0 (official windows built)
app.simplenote.com

I find there were a few issues related to the login issue, however, it seems that all of them are properly addressed and the issues were closed. Could you help check what went wrong? Thanks.

Where did you see the bug

OS: Linux
OS version: Ubuntu 18.04
Browser (if applicable):
Browser version (if applicable):
Simplenote app version: 2.9.0

codebykat commented 3 years ago

Hi @xiaofen9, the dev version runs against a demo database (which is history-analyst-dad). It's not expected that you would be able to use your production login credentials while using npm run dev. Have you tried to create a new account in the local version? (Now that I'm typing this, I am not sure that it works at all with the new, email-first signup flow.)

Are you seeing the same path being tried on 2.9.0 official linux built? That is surprising as that version should be contacting the production database.

codebykat commented 3 years ago

@xiaofen9 Just an update - after some discussion we realized the demo version of the database doesn't work with the new signup flow. We're working on a solution but it will require rethinking a few things. I'm going to rename this issue for tracking, but I'm not sure if this solves your problem -- can you let me know what you were actually trying to test or accomplish? Thanks 😄

xavierleune commented 3 years ago

Hi, for the record, here is the process I used to test this app:

Checkout the version 2.7.0 (last version with the old sign up workflow) git checkout v2.7.0
npm install && npm run dev
Create a test account inside the preview app
At this point you're logged in, so let's switch to the main branch:
git checkout develop && npm install && npm run dev

You're all set to work on the app.

DanBrown47 commented 3 years ago

Please fix this bug ASAP for further development of #2945

Chett23 commented 7 months ago

Adding on to this as I am running into this issue as well. I tried @xavierleune work around but was not able to create an account, it throws an error saying, 'unable to create account please try again'. however trying again does not fix the issue.

Please fix for So i can start to fix #3179