··· 9 ].join(' ') 10 const CLIENT_ID = !(import.meta.env.DEV) 11 ? `${window.location.origin}/oauth/client-metadata.json` 12 - : `http://localhost/oauth/client-metadata.json?scope=${SCOPES}` 13 const REDIRECT_URI = `${window.location.origin}/` 14 15 interface OAuthState {

··· 9 ].join(' ') 10 const CLIENT_ID = !(import.meta.env.DEV) 11 ? `${window.location.origin}/oauth/client-metadata.json` 12 + : `http://localhost/?scope=${SCOPES}` 13 const REDIRECT_URI = `${window.location.origin}/` 14 15 interface OAuthState {

+2 -1

src/oauth/client.rs

··· 91 url.scheme() == "http" 92 && url.host_str() == Some("localhost") 93 && url.port().is_none() 94 - && url.path().is_empty() 95 } else { 96 false 97 }

··· 91 url.scheme() == "http" 92 && url.host_str() == Some("localhost") 93 && url.port().is_none() 94 + // empty path 95 + && url.path() == "/" 96 } else { 97 false 98 }

History

1 round 1 comment

nel.pet submitted #0 2mo

1 commit

expand

d7e538e5

actually fix localhost client support

expand 1 comment

nel.pet 2mo

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