Tranquil PDS! Moved to https://tangled.org/tranquil.farm/tranquil-pds

actually fix localhost client support #4

merged opened by nel.pet targeting main from [deleted fork]: main
Labels

None yet.

assignee

None yet.

Participants 1
AT URI
at://did:plc:h5wsnqetncv6lu2weom35lg2/sh.tangled.repo.pull/3marnsyhsss22
+3 -2
Diff #0
+1 -1
frontend/src/lib/oauth.ts
··· 9 9 ].join(' ') 10 10 const CLIENT_ID = !(import.meta.env.DEV) 11 11 ? `${window.location.origin}/oauth/client-metadata.json` 12 - : `http://localhost/oauth/client-metadata.json?scope=${SCOPES}` 12 + : `http://localhost/?scope=${SCOPES}` 13 13 const REDIRECT_URI = `${window.location.origin}/` 14 14 15 15 interface OAuthState {
+2 -1
src/oauth/client.rs
··· 91 91 url.scheme() == "http" 92 92 && url.host_str() == Some("localhost") 93 93 && url.port().is_none() 94 - && url.path().is_empty() 94 + // empty path 95 + && url.path() == "/" 95 96 } else { 96 97 false 97 98 }

History

1 round 1 comment
sign up or login to add to the discussion
nel.pet submitted #0
1 commit
expand
d7e538e5
actually fix localhost client support
expand 1 comment

i frankly dont know how i forgot that an empty path is "/" and not literally empty or how i forgot to make the path actually empty for the dev mode client id but here we are

pull request successfully merged