Security hardening: headers, scanning, Docker, firewall
- Caddy: HSTS, CSP, X-Frame-Options, nosniff, Referrer-Policy, Permissions-Policy on all responses - Caddy: Block scanner paths (.env, .git, wp-config, etc.) with 403 - CI: Gitleaks secret scanning + pnpm audit for vulnerabilities - Dependabot: Weekly npm + GitHub Actions + monthly Docker updates - Docker: Non-root user in journal, planner, and brouter containers - Server: UFW firewall (22/80/443 only) + fail2ban (8 IPs already banned) - SECURITY.md: Vulnerability disclosure policy - Privacy page: Security practices section added Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
parent
0499133982
commit
be3e69c10b
11 changed files with 211 additions and 18 deletions
30
.github/dependabot.yml
vendored
Normal file
30
.github/dependabot.yml
vendored
Normal file
|
|
@ -0,0 +1,30 @@
|
||||||
|
version: 2
|
||||||
|
updates:
|
||||||
|
- package-ecosystem: "npm"
|
||||||
|
directory: "/"
|
||||||
|
schedule:
|
||||||
|
interval: "weekly"
|
||||||
|
groups:
|
||||||
|
production:
|
||||||
|
update-types:
|
||||||
|
- "minor"
|
||||||
|
- "patch"
|
||||||
|
development:
|
||||||
|
dependency-type: "development"
|
||||||
|
open-pull-requests-limit: 10
|
||||||
|
|
||||||
|
- package-ecosystem: "github-actions"
|
||||||
|
directory: "/"
|
||||||
|
schedule:
|
||||||
|
interval: "weekly"
|
||||||
|
open-pull-requests-limit: 5
|
||||||
|
|
||||||
|
- package-ecosystem: "docker"
|
||||||
|
directory: "/apps/journal"
|
||||||
|
schedule:
|
||||||
|
interval: "monthly"
|
||||||
|
|
||||||
|
- package-ecosystem: "docker"
|
||||||
|
directory: "/apps/planner"
|
||||||
|
schedule:
|
||||||
|
interval: "monthly"
|
||||||
21
.github/workflows/ci.yml
vendored
21
.github/workflows/ci.yml
vendored
|
|
@ -11,6 +11,27 @@ concurrency:
|
||||||
cancel-in-progress: true
|
cancel-in-progress: true
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
|
security:
|
||||||
|
name: Security Scan
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v6
|
||||||
|
with:
|
||||||
|
fetch-depth: 0
|
||||||
|
- name: Gitleaks
|
||||||
|
uses: gitleaks/gitleaks-action@v2
|
||||||
|
env:
|
||||||
|
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
||||||
|
- uses: pnpm/action-setup@v4
|
||||||
|
- uses: actions/setup-node@v6
|
||||||
|
with:
|
||||||
|
node-version: 24
|
||||||
|
cache: pnpm
|
||||||
|
- run: pnpm install --frozen-lockfile
|
||||||
|
- name: Dependency audit
|
||||||
|
run: pnpm audit --audit-level=high
|
||||||
|
continue-on-error: true
|
||||||
|
|
||||||
typecheck:
|
typecheck:
|
||||||
name: Typecheck
|
name: Typecheck
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
|
|
||||||
7
.gitleaks.toml
Normal file
7
.gitleaks.toml
Normal file
|
|
@ -0,0 +1,7 @@
|
||||||
|
[extend]
|
||||||
|
# Use default gitleaks rules
|
||||||
|
|
||||||
|
[[allowlist.regexTarget]]
|
||||||
|
# Sentry DSNs are public (ingest-only, not secrets)
|
||||||
|
description = "Sentry DSN"
|
||||||
|
regex = '''https://[a-f0-9]+@o\d+\.ingest\.\w+\.sentry\.io/\d+'''
|
||||||
44
SECURITY.md
Normal file
44
SECURITY.md
Normal file
|
|
@ -0,0 +1,44 @@
|
||||||
|
# Security Policy
|
||||||
|
|
||||||
|
## Reporting a Vulnerability
|
||||||
|
|
||||||
|
If you discover a security vulnerability in trails.cool, please report it
|
||||||
|
responsibly.
|
||||||
|
|
||||||
|
**Do NOT open a public GitHub issue for security vulnerabilities.**
|
||||||
|
|
||||||
|
Instead, please email: **security@trails.cool**
|
||||||
|
|
||||||
|
Include:
|
||||||
|
- Description of the vulnerability
|
||||||
|
- Steps to reproduce
|
||||||
|
- Impact assessment (what could an attacker do?)
|
||||||
|
- Any suggested fixes
|
||||||
|
|
||||||
|
We will acknowledge your report within 48 hours and provide a timeline for
|
||||||
|
a fix. We aim to resolve critical vulnerabilities within 7 days.
|
||||||
|
|
||||||
|
## Scope
|
||||||
|
|
||||||
|
The following are in scope:
|
||||||
|
- trails.cool and planner.trails.cool (production)
|
||||||
|
- The trails-cool/trails GitHub repository
|
||||||
|
- Authentication, authorization, and session management
|
||||||
|
- Data exposure or leakage
|
||||||
|
- Server-side vulnerabilities
|
||||||
|
|
||||||
|
The following are out of scope:
|
||||||
|
- Denial of service attacks
|
||||||
|
- Social engineering
|
||||||
|
- Self-hosted instances (report to the instance operator)
|
||||||
|
- Issues in third-party dependencies (report upstream)
|
||||||
|
|
||||||
|
## Security Practices
|
||||||
|
|
||||||
|
- **Authentication**: Passkey (WebAuthn) + magic link login. No passwords stored.
|
||||||
|
- **Cookies**: httpOnly, secure, sameSite=lax
|
||||||
|
- **Database**: Parameterized queries via Drizzle ORM (no raw SQL)
|
||||||
|
- **Headers**: HSTS, CSP, X-Frame-Options, X-Content-Type-Options
|
||||||
|
- **CI**: Gitleaks secret scanning, dependency auditing
|
||||||
|
- **Docker**: Non-root containers
|
||||||
|
- **Monitoring**: Sentry error tracking, fail2ban SSH protection
|
||||||
|
|
@ -21,6 +21,7 @@ COPY . .
|
||||||
RUN pnpm --filter @trails-cool/journal build
|
RUN pnpm --filter @trails-cool/journal build
|
||||||
|
|
||||||
FROM base AS runtime
|
FROM base AS runtime
|
||||||
|
RUN addgroup --system app && adduser --system --ingroup app app
|
||||||
COPY --from=deps /app/node_modules ./node_modules
|
COPY --from=deps /app/node_modules ./node_modules
|
||||||
COPY --from=deps /app/apps/journal/node_modules ./apps/journal/node_modules
|
COPY --from=deps /app/apps/journal/node_modules ./apps/journal/node_modules
|
||||||
COPY --from=build /app/apps/journal/build ./apps/journal/build
|
COPY --from=build /app/apps/journal/build ./apps/journal/build
|
||||||
|
|
@ -28,6 +29,7 @@ COPY --from=build /app/apps/journal/package.json ./apps/journal/package.json
|
||||||
COPY --from=build /app/packages ./packages
|
COPY --from=build /app/packages ./packages
|
||||||
|
|
||||||
WORKDIR /app/apps/journal
|
WORKDIR /app/apps/journal
|
||||||
|
USER app
|
||||||
EXPOSE 3000
|
EXPOSE 3000
|
||||||
ENV PORT=3000
|
ENV PORT=3000
|
||||||
CMD ["npx", "react-router-serve", "./build/server/index.js"]
|
CMD ["npx", "react-router-serve", "./build/server/index.js"]
|
||||||
|
|
|
||||||
|
|
@ -90,6 +90,18 @@ export default function PrivacyPage() {
|
||||||
</ul>
|
</ul>
|
||||||
</section>
|
</section>
|
||||||
|
|
||||||
|
<section className="mt-10">
|
||||||
|
<h2 className="text-xl font-semibold text-gray-900">Security Practices</h2>
|
||||||
|
<ul className="mt-3 list-disc pl-6 text-gray-600 space-y-1">
|
||||||
|
<li><strong>Authentication</strong>: Passkey (WebAuthn) and magic link login. No passwords stored.</li>
|
||||||
|
<li><strong>Encryption</strong>: All traffic over HTTPS with HSTS preload. Cookies are httpOnly and secure.</li>
|
||||||
|
<li><strong>Headers</strong>: Content-Security-Policy, X-Frame-Options, X-Content-Type-Options on all responses.</li>
|
||||||
|
<li><strong>Infrastructure</strong>: Docker containers run as non-root. Firewall restricts to HTTP/HTTPS/SSH only.</li>
|
||||||
|
<li><strong>CI/CD</strong>: Gitleaks secret scanning and dependency auditing on every pull request.</li>
|
||||||
|
<li><strong>Vulnerability reporting</strong>: See our <a href="https://github.com/trails-cool/trails/blob/main/SECURITY.md" className="text-blue-600 hover:underline">SECURITY.md</a> for responsible disclosure.</li>
|
||||||
|
</ul>
|
||||||
|
</section>
|
||||||
|
|
||||||
<p className="mt-10 text-sm text-gray-500">
|
<p className="mt-10 text-sm text-gray-500">
|
||||||
Last updated: March 2026. If this manifest changes, we'll note it here.
|
Last updated: March 2026. If this manifest changes, we'll note it here.
|
||||||
</p>
|
</p>
|
||||||
|
|
|
||||||
|
|
@ -21,6 +21,7 @@ COPY . .
|
||||||
RUN pnpm --filter @trails-cool/planner build
|
RUN pnpm --filter @trails-cool/planner build
|
||||||
|
|
||||||
FROM base AS runtime
|
FROM base AS runtime
|
||||||
|
RUN addgroup --system app && adduser --system --ingroup app app
|
||||||
COPY --from=deps /app/node_modules ./node_modules
|
COPY --from=deps /app/node_modules ./node_modules
|
||||||
COPY --from=deps /app/apps/planner/node_modules ./apps/planner/node_modules
|
COPY --from=deps /app/apps/planner/node_modules ./apps/planner/node_modules
|
||||||
COPY --from=build /app/apps/planner/build ./apps/planner/build
|
COPY --from=build /app/apps/planner/build ./apps/planner/build
|
||||||
|
|
@ -30,6 +31,7 @@ COPY --from=build /app/apps/planner/package.json ./apps/planner/package.json
|
||||||
COPY --from=build /app/packages ./packages
|
COPY --from=build /app/packages ./packages
|
||||||
|
|
||||||
WORKDIR /app/apps/planner
|
WORKDIR /app/apps/planner
|
||||||
|
USER app
|
||||||
EXPOSE 3001
|
EXPOSE 3001
|
||||||
ENV PORT=3001
|
ENV PORT=3001
|
||||||
CMD ["node", "--experimental-strip-types", "server.ts"]
|
CMD ["node", "--experimental-strip-types", "server.ts"]
|
||||||
|
|
|
||||||
|
|
@ -12,13 +12,15 @@ RUN apt-get update && apt-get install -y --no-install-recommends wget unzip curl
|
||||||
&& rm "brouter-${BROUTER_VERSION}.zip" \
|
&& rm "brouter-${BROUTER_VERSION}.zip" \
|
||||||
&& apt-get purge -y wget unzip && apt-get autoremove -y && rm -rf /var/lib/apt/lists/*
|
&& apt-get purge -y wget unzip && apt-get autoremove -y && rm -rf /var/lib/apt/lists/*
|
||||||
|
|
||||||
# Create directories for segments and profiles
|
# Create non-root user and directories for segments and profiles
|
||||||
RUN mkdir -p /data/segments /data/profiles
|
RUN addgroup --system app && adduser --system --ingroup app app \
|
||||||
|
&& mkdir -p /data/segments /data/profiles
|
||||||
|
|
||||||
# Copy default profiles from release and rename JAR for simpler CMD
|
# Copy default profiles from release and rename JAR for simpler CMD
|
||||||
RUN cp -r profiles2/* /data/profiles/ 2>/dev/null || true \
|
RUN cp -r profiles2/* /data/profiles/ 2>/dev/null || true \
|
||||||
&& mv brouter-*-all.jar brouter.jar
|
&& mv brouter-*-all.jar brouter.jar
|
||||||
|
|
||||||
|
USER app
|
||||||
EXPOSE 17777
|
EXPOSE 17777
|
||||||
|
|
||||||
# BRouter server: <segmentdir> <profiledir> <customprofiledir> <port> <maxthreads>
|
# BRouter server: <segmentdir> <profiledir> <customprofiledir> <port> <maxthreads>
|
||||||
|
|
|
||||||
53
docs/server-hardening.md
Normal file
53
docs/server-hardening.md
Normal file
|
|
@ -0,0 +1,53 @@
|
||||||
|
# Server Hardening
|
||||||
|
|
||||||
|
Steps to harden the Hetzner CX21 production server.
|
||||||
|
|
||||||
|
## UFW Firewall
|
||||||
|
|
||||||
|
```bash
|
||||||
|
# Install and enable
|
||||||
|
apt install -y ufw
|
||||||
|
ufw default deny incoming
|
||||||
|
ufw default allow outgoing
|
||||||
|
ufw allow 22/tcp # SSH
|
||||||
|
ufw allow 80/tcp # HTTP (Caddy redirect)
|
||||||
|
ufw allow 443/tcp # HTTPS
|
||||||
|
ufw enable
|
||||||
|
ufw status
|
||||||
|
```
|
||||||
|
|
||||||
|
## Fail2ban (SSH protection)
|
||||||
|
|
||||||
|
```bash
|
||||||
|
# Install
|
||||||
|
apt install -y fail2ban
|
||||||
|
|
||||||
|
# Create config
|
||||||
|
cat > /etc/fail2ban/jail.local << 'EOF'
|
||||||
|
[sshd]
|
||||||
|
enabled = true
|
||||||
|
port = ssh
|
||||||
|
filter = sshd
|
||||||
|
logpath = /var/log/auth.log
|
||||||
|
maxretry = 5
|
||||||
|
bantime = 3600
|
||||||
|
findtime = 600
|
||||||
|
EOF
|
||||||
|
|
||||||
|
# Start
|
||||||
|
systemctl enable fail2ban
|
||||||
|
systemctl start fail2ban
|
||||||
|
|
||||||
|
# Check status
|
||||||
|
fail2ban-client status sshd
|
||||||
|
```
|
||||||
|
|
||||||
|
## SSH Hardening
|
||||||
|
|
||||||
|
Already in place:
|
||||||
|
- Key-based auth only (password auth disabled by Hetzner cloud-init)
|
||||||
|
- Root login via SSH key only
|
||||||
|
|
||||||
|
Optional improvements:
|
||||||
|
- Change SSH port (security through obscurity, mild benefit)
|
||||||
|
- Add `AllowUsers root` to `/etc/ssh/sshd_config` to restrict SSH users
|
||||||
|
|
@ -1,4 +1,22 @@
|
||||||
|
(security_headers) {
|
||||||
|
header {
|
||||||
|
Strict-Transport-Security "max-age=31536000; includeSubDomains; preload"
|
||||||
|
X-Content-Type-Options "nosniff"
|
||||||
|
X-Frame-Options "DENY"
|
||||||
|
Referrer-Policy "strict-origin-when-cross-origin"
|
||||||
|
Permissions-Policy "camera=(), microphone=(), geolocation=()"
|
||||||
|
Content-Security-Policy "default-src 'self'; script-src 'self' 'unsafe-inline'; style-src 'self' 'unsafe-inline'; img-src 'self' data: https://*.tile.openstreetmap.org; connect-src 'self' wss: https://*.sentry.io https://*.ingest.de.sentry.io; font-src 'self';"
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
(block_scanners) {
|
||||||
|
@scanners path /.env* /.git* /wp-* /wp-admin* /admin* /config.* /backup* /.aws* /.docker* /composer* /vendor* *.php *.asp
|
||||||
|
respond @scanners 403
|
||||||
|
}
|
||||||
|
|
||||||
{$DOMAIN:trails.cool} {
|
{$DOMAIN:trails.cool} {
|
||||||
|
import security_headers
|
||||||
|
import block_scanners
|
||||||
reverse_proxy journal:3000
|
reverse_proxy journal:3000
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
@ -7,5 +25,7 @@ www.{$DOMAIN:trails.cool} {
|
||||||
}
|
}
|
||||||
|
|
||||||
planner.{$DOMAIN:trails.cool} {
|
planner.{$DOMAIN:trails.cool} {
|
||||||
|
import security_headers
|
||||||
|
import block_scanners
|
||||||
reverse_proxy planner:3001
|
reverse_proxy planner:3001
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -1,44 +1,44 @@
|
||||||
## 1. Caddy Security Headers
|
## 1. Caddy Security Headers
|
||||||
|
|
||||||
- [ ] 1.1 Add security headers block to Caddyfile: HSTS, X-Content-Type-Options, X-Frame-Options, Referrer-Policy, Permissions-Policy
|
- [x] 1.1 Add security headers block to Caddyfile: HSTS, X-Content-Type-Options, X-Frame-Options, Referrer-Policy, Permissions-Policy
|
||||||
- [ ] 1.2 Add Content-Security-Policy header (permissive initial policy allowing React Router inline scripts, OSM tiles, Sentry, WebSocket)
|
- [x] 1.2 Add Content-Security-Policy header (permissive initial policy allowing React Router inline scripts, OSM tiles, Sentry, WebSocket)
|
||||||
- [ ] 1.3 Test headers with securityheaders.com or curl after deploy
|
- [ ] 1.3 Test headers with securityheaders.com or curl after deploy
|
||||||
|
|
||||||
## 2. Scanner Blocking
|
## 2. Scanner Blocking
|
||||||
|
|
||||||
- [ ] 2.1 Add Caddy path matcher for common scanner targets (.env*, .git*, wp-*, admin*, config.*, backup*)
|
- [x] 2.1 Add Caddy path matcher for common scanner targets (.env*, .git*, wp-*, admin*, config.*, backup*)
|
||||||
- [ ] 2.2 Return 403 for matched paths before reverse_proxy
|
- [x] 2.2 Return 403 for matched paths before reverse_proxy
|
||||||
- [ ] 2.3 Verify scanner paths return 403, normal paths still work
|
- [ ] 2.3 Verify scanner paths return 403, normal paths still work
|
||||||
|
|
||||||
## 3. CI Secret Scanning
|
## 3. CI Secret Scanning
|
||||||
|
|
||||||
- [ ] 3.1 Add gitleaks/gitleaks-action@v2 step to CI workflow
|
- [x] 3.1 Add gitleaks/gitleaks-action@v2 step to CI workflow
|
||||||
- [ ] 3.2 Create .gitleaks.toml with allowlist for Sentry DSNs (public values)
|
- [x] 3.2 Create .gitleaks.toml with allowlist for Sentry DSNs (public values)
|
||||||
- [ ] 3.3 Verify gitleaks passes on current codebase
|
- [ ] 3.3 Verify gitleaks passes on current codebase
|
||||||
|
|
||||||
## 4. Dependency Auditing
|
## 4. Dependency Auditing
|
||||||
|
|
||||||
- [ ] 4.1 Add `pnpm audit --audit-level=high` step to CI workflow
|
- [x] 4.1 Add `pnpm audit --audit-level=high` step to CI workflow
|
||||||
- [ ] 4.2 Create .github/dependabot.yml for weekly pnpm dependency updates
|
- [x] 4.2 Create .github/dependabot.yml for weekly pnpm dependency updates
|
||||||
- [ ] 4.3 Verify pnpm audit passes on current codebase
|
- [ ] 4.3 Verify pnpm audit passes on current codebase
|
||||||
|
|
||||||
## 5. Docker Hardening
|
## 5. Docker Hardening
|
||||||
|
|
||||||
- [ ] 5.1 Add non-root user to Journal Dockerfile runtime stage
|
- [x] 5.1 Add non-root user to Journal Dockerfile runtime stage
|
||||||
- [ ] 5.2 Add non-root user to Planner Dockerfile runtime stage
|
- [x] 5.2 Add non-root user to Planner Dockerfile runtime stage
|
||||||
- [ ] 5.3 Add non-root user to BRouter Dockerfile
|
- [x] 5.3 Add non-root user to BRouter Dockerfile
|
||||||
- [ ] 5.4 Test containers start and serve correctly as non-root
|
- [ ] 5.4 Test containers start and serve correctly as non-root
|
||||||
|
|
||||||
## 6. Server Hardening
|
## 6. Server Hardening
|
||||||
|
|
||||||
- [ ] 6.1 Install and configure fail2ban on Hetzner server (SSH protection)
|
- [x] 6.1 Install and configure fail2ban on Hetzner server (SSH protection)
|
||||||
- [ ] 6.2 Configure UFW firewall (allow 22, 80, 443 only)
|
- [x] 6.2 Configure UFW firewall (allow 22, 80, 443 only)
|
||||||
- [ ] 6.3 Document server hardening steps in deployment docs
|
- [x] 6.3 Document server hardening steps in deployment docs
|
||||||
|
|
||||||
## 7. Documentation
|
## 7. Documentation
|
||||||
|
|
||||||
- [ ] 7.1 Create SECURITY.md with vulnerability disclosure policy and security contact
|
- [x] 7.1 Create SECURITY.md with vulnerability disclosure policy and security contact
|
||||||
- [ ] 7.2 Update privacy manifest with security practices documented
|
- [x] 7.2 Update privacy manifest with security practices documented
|
||||||
|
|
||||||
## 8. Verify
|
## 8. Verify
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue