Skip to content

refactor(config): consolidate origin validation - #15

Closed
thomasorgeval wants to merge 1 commit into
mainfrom
small-refactor/config-origin-validation
Closed

refactor(config): consolidate origin validation#15
thomasorgeval wants to merge 1 commit into
mainfrom
small-refactor/config-origin-validation

Conversation

@thomasorgeval

Copy link
Copy Markdown
Member

Summary

  • Consolidates config origin parsing/HTTPS validation into one helper shared by trusted origins and APP_PUBLIC_URL.
  • Keeps the existing accepted origins and validation messages behavior unchanged.

Expected impact

  • Reduces duplicated URL parsing and local-HTTP exception checks in config validation.
  • Makes future origin validation changes less error-prone and easier to review.

Verification

  • pnpm nx test config
  • pnpm nx typecheck config
  • pnpm nx lint config
  • pnpm format:check -- packages/config/src/index.ts
  • git diff --check

@thomasorgeval
thomasorgeval deleted the small-refactor/config-origin-validation branch June 26, 2026 14:13
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.

1 participant