Skip to content

fix: resolve all 16 GitHub issues - #18

Merged
ralyodio merged 2 commits into
masterfrom
fix/github-issues
Mar 19, 2026
Merged

fix: resolve all 16 GitHub issues#18
ralyodio merged 2 commits into
masterfrom
fix/github-issues

Conversation

@ralyodio

Copy link
Copy Markdown
Contributor

Fixes all open issues:

Critical/High:

Medium:

Low:

Closed as intended:

Closes #1 #2 #3 #4 #5 #6 #7 #8 #9 #10 #11 #12 #13 #14 #15 #16

#1 - Static files (manifest, robots, sitemap) no longer redirect to /login
#2 - Security headers (CSP, X-Frame-Options, HSTS, etc.)
#4 - All /api routes are public (no auth redirect)
#5 - Added og:image and twitter:image meta tags
#6 - Removed duplicate mobile-web-app-capable meta
#7 - Password min 8 chars on login/register
#8 - Forgot password link + page with Supabase integration
#10 - Mobile hamburger menu in header
#15 - Business plan CTA → mailto:sales@smshub.dev
#16 - Health endpoint covered by /api public route

Closes #1 #2 #3 #4 #5 #6 #7 #8 #9 #10 #11 #12 #13 #14 #15 #16
- Health endpoint returns 200 with status ok
- Public routes include static files, API, and public pages
- Security headers present in next.config.ts
- 108 tests total (was 103)
@ralyodio
ralyodio merged commit 0838b7e into master Mar 19, 2026
4 checks passed
@ralyodio
ralyodio deleted the fix/github-issues branch March 19, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Critical] manifest.json, robots.txt, and sitemap.xml redirect to /login instead of serving static content

1 participant