fix: harden production login and security headers
This commit is contained in:
@@ -44,6 +44,35 @@ test.describe('authenticated route and deep-link smoke', () => {
|
|||||||
await expect(page.getByRole('button', { name: 'Anmelden' })).toBeVisible()
|
await expect(page.getByRole('button', { name: 'Anmelden' })).toBeVisible()
|
||||||
})
|
})
|
||||||
|
|
||||||
|
test('does not present the rolling login window as a lock while attempts remain', async ({ page }) => {
|
||||||
|
await mockNexusApi(page, { authenticated: false })
|
||||||
|
await page.route('**/api/v1/auth/login', async route => {
|
||||||
|
await route.fulfill({
|
||||||
|
status: 401,
|
||||||
|
contentType: 'application/problem+json',
|
||||||
|
headers: {
|
||||||
|
'X-RateLimit-Remaining': '4',
|
||||||
|
'X-RateLimit-Reset': String(Math.ceil(Date.now() / 1000) + 60),
|
||||||
|
},
|
||||||
|
body: JSON.stringify({
|
||||||
|
message: 'Invalid email or password.',
|
||||||
|
remaining: 4,
|
||||||
|
retryAfterSeconds: 60,
|
||||||
|
}),
|
||||||
|
})
|
||||||
|
})
|
||||||
|
|
||||||
|
await page.goto('/login')
|
||||||
|
await page.getByLabel('E-Mail').fill('release-smoke@example.invalid')
|
||||||
|
await page.getByLabel('Passwort', { exact: true }).fill('not-a-real-password')
|
||||||
|
await page.getByRole('button', { name: 'Anmelden' }).click()
|
||||||
|
|
||||||
|
await expect(page.getByRole('alert')).toContainText('4 Versuche verbleibend')
|
||||||
|
await expect(page.getByText(/Entsperrt in/)).toHaveCount(0)
|
||||||
|
await expect(page.getByRole('button', { name: 'Anmelden' })).toBeEnabled()
|
||||||
|
await expect(page.getByRole('button', { name: /Gesperrt/ })).toHaveCount(0)
|
||||||
|
})
|
||||||
|
|
||||||
test('renders the core owner deep links without a client-side exception', async ({ page }) => {
|
test('renders the core owner deep links without a client-side exception', async ({ page }) => {
|
||||||
test.setTimeout(90_000)
|
test.setTimeout(90_000)
|
||||||
await mockNexusApi(page, { role: 'owner' })
|
await mockNexusApi(page, { role: 'owner' })
|
||||||
|
|||||||
+17
-5
@@ -5,16 +5,16 @@ server {
|
|||||||
root /usr/share/nginx/html;
|
root /usr/share/nginx/html;
|
||||||
index index.html;
|
index index.html;
|
||||||
|
|
||||||
|
# Gehashte Assets: 1 Jahr cachen (immutable wg. Content-Hash im Dateinamen)
|
||||||
|
location /assets/ {
|
||||||
|
expires 1y;
|
||||||
|
add_header Cache-Control "public, immutable";
|
||||||
|
add_header Strict-Transport-Security "max-age=31536000; includeSubDomains" always;
|
||||||
add_header Content-Security-Policy "default-src 'self'; connect-src 'self'; img-src 'self' data:; script-src 'self'; style-src 'self' 'unsafe-inline'; frame-ancestors 'none'; base-uri 'self'; form-action 'self'" always;
|
add_header Content-Security-Policy "default-src 'self'; connect-src 'self'; img-src 'self' data:; script-src 'self'; style-src 'self' 'unsafe-inline'; frame-ancestors 'none'; base-uri 'self'; form-action 'self'" always;
|
||||||
add_header Referrer-Policy "no-referrer" always;
|
add_header Referrer-Policy "no-referrer" always;
|
||||||
add_header X-Content-Type-Options "nosniff" always;
|
add_header X-Content-Type-Options "nosniff" always;
|
||||||
add_header X-Frame-Options "DENY" always;
|
add_header X-Frame-Options "DENY" always;
|
||||||
add_header Permissions-Policy "camera=(), microphone=(), geolocation=()" always;
|
add_header Permissions-Policy "camera=(), microphone=(), geolocation=()" always;
|
||||||
|
|
||||||
# Gehashte Assets: 1 Jahr cachen (immutable wg. Content-Hash im Dateinamen)
|
|
||||||
location /assets/ {
|
|
||||||
expires 1y;
|
|
||||||
add_header Cache-Control "public, immutable";
|
|
||||||
}
|
}
|
||||||
|
|
||||||
# SPA-Entry nie cachen
|
# SPA-Entry nie cachen
|
||||||
@@ -22,6 +22,12 @@ server {
|
|||||||
add_header Cache-Control "no-cache, no-store, must-revalidate";
|
add_header Cache-Control "no-cache, no-store, must-revalidate";
|
||||||
add_header Pragma "no-cache";
|
add_header Pragma "no-cache";
|
||||||
add_header Expires "0";
|
add_header Expires "0";
|
||||||
|
add_header Strict-Transport-Security "max-age=31536000; includeSubDomains" always;
|
||||||
|
add_header Content-Security-Policy "default-src 'self'; connect-src 'self'; img-src 'self' data:; script-src 'self'; style-src 'self' 'unsafe-inline'; frame-ancestors 'none'; base-uri 'self'; form-action 'self'" always;
|
||||||
|
add_header Referrer-Policy "no-referrer" always;
|
||||||
|
add_header X-Content-Type-Options "nosniff" always;
|
||||||
|
add_header X-Frame-Options "DENY" always;
|
||||||
|
add_header Permissions-Policy "camera=(), microphone=(), geolocation=()" always;
|
||||||
}
|
}
|
||||||
|
|
||||||
# Bridge-Endpunkte (Agent-zu-Backend): separater Pfad ohne CSP-Einschränkungen
|
# Bridge-Endpunkte (Agent-zu-Backend): separater Pfad ohne CSP-Einschränkungen
|
||||||
@@ -66,6 +72,12 @@ server {
|
|||||||
|
|
||||||
location / {
|
location / {
|
||||||
try_files $uri $uri/ /index.html;
|
try_files $uri $uri/ /index.html;
|
||||||
|
add_header Strict-Transport-Security "max-age=31536000; includeSubDomains" always;
|
||||||
|
add_header Content-Security-Policy "default-src 'self'; connect-src 'self'; img-src 'self' data:; script-src 'self'; style-src 'self' 'unsafe-inline'; frame-ancestors 'none'; base-uri 'self'; form-action 'self'" always;
|
||||||
|
add_header Referrer-Policy "no-referrer" always;
|
||||||
|
add_header X-Content-Type-Options "nosniff" always;
|
||||||
|
add_header X-Frame-Options "DENY" always;
|
||||||
|
add_header Permissions-Policy "camera=(), microphone=(), geolocation=()" always;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -65,8 +65,10 @@ async function submit() {
|
|||||||
if (reason instanceof Error) {
|
if (reason instanceof Error) {
|
||||||
error.value = reason.message
|
error.value = reason.message
|
||||||
|
|
||||||
// Start countdown if rate-limited
|
// The reset timestamp also describes the rolling attempt window after a
|
||||||
if (auth.retryAfterSeconds > 0) {
|
// regular 401. Only present a lock countdown once the backend has
|
||||||
|
// actually exhausted the remaining attempts.
|
||||||
|
if (auth.isRateLimited && auth.retryAfterSeconds > 0) {
|
||||||
startCountdown(auth.retryAfterSeconds)
|
startCountdown(auth.retryAfterSeconds)
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
@@ -152,14 +154,14 @@ async function submit() {
|
|||||||
<!-- Error display with remaining attempts -->
|
<!-- Error display with remaining attempts -->
|
||||||
<div v-if="error" class="error-box" role="alert">
|
<div v-if="error" class="error-box" role="alert">
|
||||||
<div class="error-main">
|
<div class="error-main">
|
||||||
<AlertTriangle v-if="countdown > 0" :size="16" class="error-icon" />
|
<AlertTriangle v-if="auth.isRateLimited && countdown > 0" :size="16" class="error-icon" />
|
||||||
<span>{{ error }}</span>
|
<span>{{ error }}</span>
|
||||||
</div>
|
</div>
|
||||||
<div v-if="auth.remainingAttempts !== null && auth.remainingAttempts > 0" class="attempts-remaining">
|
<div v-if="auth.remainingAttempts !== null && auth.remainingAttempts > 0" class="attempts-remaining">
|
||||||
<LockKeyhole :size="12" />
|
<LockKeyhole :size="12" />
|
||||||
<span>{{ auth.remainingAttempts }} {{ auth.remainingAttempts === 1 ? 'Versuch verbleibend' : 'Versuche verbleibend' }}</span>
|
<span>{{ auth.remainingAttempts }} {{ auth.remainingAttempts === 1 ? 'Versuch verbleibend' : 'Versuche verbleibend' }}</span>
|
||||||
</div>
|
</div>
|
||||||
<div v-if="countdown > 0" class="countdown-bar">
|
<div v-if="auth.isRateLimited && countdown > 0" class="countdown-bar">
|
||||||
<Clock :size="12" />
|
<Clock :size="12" />
|
||||||
<span>Entsperrt in {{ countdownText }}</span>
|
<span>Entsperrt in {{ countdownText }}</span>
|
||||||
</div>
|
</div>
|
||||||
@@ -168,7 +170,7 @@ async function submit() {
|
|||||||
<button type="submit" class="submit-btn" :disabled="auth.loading || !email || !password || auth.isRateLimited">
|
<button type="submit" class="submit-btn" :disabled="auth.loading || !email || !password || auth.isRateLimited">
|
||||||
<LockKeyhole :size="15" />
|
<LockKeyhole :size="15" />
|
||||||
<template v-if="auth.loading">Anmelden…</template>
|
<template v-if="auth.loading">Anmelden…</template>
|
||||||
<template v-else-if="countdown > 0">Gesperrt ({{ countdownText }})</template>
|
<template v-else-if="auth.isRateLimited && countdown > 0">Gesperrt ({{ countdownText }})</template>
|
||||||
<template v-else>Anmelden</template>
|
<template v-else>Anmelden</template>
|
||||||
</button>
|
</button>
|
||||||
</form>
|
</form>
|
||||||
|
|||||||
Reference in New Issue
Block a user