Fix account sign in on the PDS #4

merged opened by

willdot.net 5 months ago targeting main from willdot.net/pds-gatekeeper: fix/account-sign-in

This fixes https://tangled.org/@baileytownsend.dev/pds-gatekeeper/issues/3 as described in the comments

Tested by building the app locally, cross compiling for Linux and copying to my PDS server and running the app instead of the latest docker image.

When doing this I was able to access the /account/<did> page by signing in.

Labels

None yet.

Participants 2

AT URI

at://did:plc:dadhhalkfcq3gucaq25hjqon/sh.tangled.repo.pull/3m2ffpgc7lu22

+2 -1

1 changed file

Diff round #0

expand all

unified split

src

oauth_provider.rs

+2 -1

src/oauth_provider.rs

··· 13 pub struct SignInRequest { 14 pub username: String, 15 pub password: String, 16 - pub remember: bool, 17 pub locale: String, 18 #[serde(skip_serializing_if = "Option::is_none", rename = "emailOtp")] 19 pub email_otp: Option<String>,

··· 13 pub struct SignInRequest { 14 pub username: String, 15 pub password: String, 16 + #[serde(skip_serializing_if = "Option::is_none")] 17 + pub remember: Option<bool>, 18 pub locale: String, 19 #[serde(skip_serializing_if = "Option::is_none", rename = "emailOtp")] 20 pub email_otp: Option<String>,

History

1 round 1 comment

willdot.net submitted #0 5mo

1 commit

expand

c77c69a9

make the remember field optional

expand 1 comment

baileytownsend.dev 5mo

Looks good! Thank you so much!

pull request successfully merged