Skip to content

feat: validate and normalize input data#36

Merged
Vtec234 merged 15 commits into
mainfrom
project-page
Jun 5, 2026
Merged

feat: validate and normalize input data#36
Vtec234 merged 15 commits into
mainfrom
project-page

Conversation

@Vtec234

@Vtec234 Vtec234 commented Jun 5, 2026

Copy link
Copy Markdown
Member
  • Enforce stricter constraints on project/user/template names/IDs - ASCII and no special characters besides hyphens and underscores; see the diff in src/lib/util.ts.
  • Consistently validate input data across server functions and authentication code.
  • Compare user/project names up to casing in SQL. Fixes Enforce ASCII-only, case-insensitive user and project names #23.

Also a few misc changes:

  • Factor out Error component.
  • Replace node:fs with node:fs/promises in user profile actions.
  • Return 404 instead of a rendered error in project pages with incorrect URL.
  • Remove old test scripts.

@Vtec234 Vtec234 requested a review from jcreedcmu June 5, 2026 14:01
@Vtec234 Vtec234 merged commit 3f5f021 into main Jun 5, 2026
1 check passed
@Vtec234 Vtec234 deleted the project-page branch June 5, 2026 17:08
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.

Enforce ASCII-only, case-insensitive user and project names

2 participants