versia-pub/server
2
0
Fork 0
mirror of https://github.com/versia-pub/server.git synced 2025-12-06 16:38:19 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

feat(api): 🐛 Change consent route from /oauth/redirect to /oauth/consent

Browse source
This commit is contained in:
Jesse Wierzbinski 2024-04-21 21:36:45 -10:00
parent 99822a716d
commit 9834825184
No known key found for this signature in database
3 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
server/api/api/auth/login/index.ts
View file

@ -150,7 +150,7 @@ export default apiRoute(async (req, matchedRoute, extraData) => {
// Redirect to OAuth authorize with JWT // Redirect to OAuth authorize with JWT
return response(null, 302, { return response(null, 302, {
Location: new URL( Location: new URL(
`/oauth/redirect?${searchParams.toString()}`, `/oauth/consent?${searchParams.toString()}`,
config.http.base_url, config.http.base_url,
).toString(), ).toString(),
// Set cookie with JWT // Set cookie with JWT

4
server/api/oauth/callback/[issuer]/index.ts
View file

@ -177,8 +177,8 @@ export default apiRoute(async (req, matchedRoute, extraData) => {
// Redirect back to application // Redirect back to application
return Response.redirect( return Response.redirect(
`/oauth/redirect?${new URLSearchParams({ `/oauth/consent?${new URLSearchParams({
redirect_uri: flow.application.redirectUris, redirect_uri: flow.application.redirectUri,
code, code,
client_id: flow.application.clientId, client_id: flow.application.clientId,
application: flow.application.name, application: flow.application.name,

2
tests/oauth.test.ts
View file

@ -84,7 +84,7 @@ describe("POST /api/auth/login/", () => {
"", "",
); );
expect(locationHeader.pathname).toBe("/oauth/redirect"); expect(locationHeader.pathname).toBe("/oauth/consent");
expect(locationHeader.searchParams.get("client_id")).toBe(client_id); expect(locationHeader.searchParams.get("client_id")).toBe(client_id);
expect(locationHeader.searchParams.get("redirect_uri")).toBe( expect(locationHeader.searchParams.get("redirect_uri")).toBe(
"https://example.com", "https://example.com",