xiaoju/ograph
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(engine): return 204 for /favicon.ico instead of 401

Browse Source 
Browser requests to /favicon.ico were hitting the auth middleware.
Added explicit route + auth bypass.
This commit is contained in:
小糯 🐱 2026-04-13 17:14:45 +08:00
parent bd4b79bd7b
commit 5cc46b3e9b
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
packages/engine/src/index.ts
View File

@ -117,6 +117,10 @@ app.use('*', async (c, next) => {
// UI (no auth, served before auth middleware) // UI (no auth, served before auth middleware)
// ============================================ // ============================================
app.get('/favicon.ico', (c) => {
return new Response(null, { status: 204 })
})
app.get('/ui', (c) => { app.get('/ui', (c) => {
return c.html(UI_HTML) return c.html(UI_HTML)
}) })
@ -143,7 +147,7 @@ app.get('/schema', async (c) => {
// Auth middleware for all routes except health, schema, ui, and POST /events (which has its own dual auth) // Auth middleware for all routes except health, schema, ui, and POST /events (which has its own dual auth)
app.use('*', async (c, next) => { app.use('*', async (c, next) => {
if (c.req.path === '/health' || c.req.path === '/schema' || c.req.path.startsWith('/ui')) return next() if (c.req.path === '/health' || c.req.path === '/schema' || c.req.path === '/favicon.ico' || c.req.path.startsWith('/ui')) return next()
if (c.req.method === 'POST' && c.req.path === '/events') return next() if (c.req.method === 'POST' && c.req.path === '/events') return next()
return bearerAuth(c.env.API_TOKEN)(c, next) return bearerAuth(c.env.API_TOKEN)(c, next)
}) })