jeffemmett
/
rspace-online
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(notifications): use full URL pathname for proxy path 

c.req.path in Hono returns the full path, not the relative path
within the sub-router, causing the /api/notifications prefix to
be doubled.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Jeff Emmett 2026-03-23 13:36:51 -07:00
parent cb784b8102
commit 023a9e7fbd
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
server/notification-routes.ts
View File

@ -13,9 +13,8 @@ export const notificationRouter = new Hono();
// Proxy all notification requests to encryptid
notificationRouter.all("/*", async (c) => {
const path = `/api/notifications${c.req.path === "/" ? "" : c.req.path}`;
const search = new URL(c.req.url).search;
const targetUrl = `${ENCRYPTID_URL}${path}${search}`;
const url = new URL(c.req.url);
const targetUrl = `${ENCRYPTID_URL}${url.pathname}${url.search}`;
const headers = new Headers(c.req.raw.headers);
headers.delete("host");
@ -32,8 +31,8 @@ notificationRouter.all("/*", async (c) => {
} catch (e: any) {
console.error("[notifications proxy] Failed to reach encryptid:", e?.message);
// Return safe fallbacks instead of hanging
if (c.req.path === "/count") return c.json({ unreadCount: 0 });
if (c.req.path === "/") return c.json({ notifications: [] });
if (url.pathname.endsWith("/count")) return c.json({ unreadCount: 0 });
if (url.pathname === "/api/notifications") return c.json({ notifications: [] });
return c.json({ error: "Notification service unavailable" }, 503);
}
});