Closed odyniec closed 1 year ago
In line 358 of oauth2.ts, it looks like getQuery() can't parse route.hash. The resulting hash is an empty object. It seems getQuery() only parses queries starting with ?, as I was able to check with a simple test:
oauth2.ts
getQuery()
route.hash
hash
?
const query1 = "?foo=bar&baz=xyzzy" console.log(getQuery(query1)) // { foo: 'bar', baz: 'xyzzy' } console.log(getQuery(query1.slice(1))) // {} const query2 = "#foo=bar&baz=xyzzy" console.log(getQuery(query2)) // {} console.log(getQuery(query2.slice(1))) // {}
parseQuery() on the other hand seems to work -- so maybe oauth2.ts just needs the same treatment as openIDConnect.ts got here?
parseQuery()
openIDConnect.ts
In line 358 of
oauth2.ts
, it looks likegetQuery()
can't parseroute.hash
. The resultinghash
is an empty object. It seemsgetQuery()
only parses queries starting with?
, as I was able to check with a simple test:parseQuery()
on the other hand seems to work -- so maybeoauth2.ts
just needs the same treatment asopenIDConnect.ts
got here?