hotel-manage-stack/auth
1
0
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity Actions

Cookie: Secure only over HTTPS (works on LAN HTTP); healthcheck 127.0.0.1

Browse source
This commit is contained in:
jtricerolph 2026-07-01 14:42:28 +00:00
parent e194e736d0
commit b7589b702a
2 changed files with 9 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

2
docker-compose.yml
View file

@ -15,7 +15,7 @@ services:
ports:
- "3001:3001"
healthcheck:
test: ["CMD-SHELL", "wget -qO- http://localhost:3001/health || exit 1"]
test: ["CMD-SHELL", "wget -qO- http://127.0.0.1:3001/health || exit 1"]
interval: 10s
retries: 5
start_period: 20s

12
src/routes/auth.js
View file

@ -6,10 +6,14 @@ const DOMAIN = process.env.DOMAIN || 'localhost'
const SESSION_DAYS = parseInt(process.env.SESSION_DAYS || '30')
const COOKIE_MAX_AGE = SESSION_DAYS * 24 * 60 * 60
function cookieOpts(clear = false) {
function cookieOpts(request, clear = false) {
// Mark the cookie Secure only when the request actually arrived over HTTPS
// (via NPM's X-Forwarded-Proto). Over plain HTTP on the LAN, a Secure cookie
// is silently dropped by the browser — so adapt to the real scheme.
const proto = request?.headers?.['x-forwarded-proto'] || request?.protocol
return {
httpOnly: true,
secure: process.env.NODE_ENV !== 'development',
secure: proto === 'https',
sameSite: 'lax',
domain: DOMAIN === 'localhost' ? undefined : `.${DOMAIN}`,
path: '/',
@ -61,13 +65,13 @@ export async function authRoutes(app) {
apps: full.apps.map(a => a.slug),
})
reply.setCookie('hnf_session', token, cookieOpts())
reply.setCookie('hnf_session', token, cookieOpts(request))
return full
})
// POST /api/auth/logout
app.post('/logout', async (request, reply) => {
reply.clearCookie('hnf_session', cookieOpts(true))
reply.clearCookie('hnf_session', cookieOpts(request, true))
return { ok: true }
})