Skip to content

fix(installer): make URL validation locale-independent - #16

Open
OiAnthony wants to merge 1 commit into
OtterMind:mainfrom
OiAnthony:fix/issue-15-locale-url-validation
Open

OiAnthony wants to merge 1 commit into
OtterMind:mainfrom
OiAnthony:fix/issue-15-locale-url-validation

Conversation

@OiAnthony

Copy link
Copy Markdown

Summary

  • force the POSIX installer to use the C locale for ASCII validation
  • prevent macOS locale collation from treating lowercase host characters as matches for [A-Z]
  • add an end-to-end installer regression test under en_US.UTF-8

Verification

  • cargo test --test install_cli_script
  • cargo fmt -- --check
  • sh -n scripts/install-cli.sh

Closes #15

Set LC_ALL=C before ASCII URL validation so macOS locale collation cannot reject lowercase hosts such as github.com.\n\nCloses OtterMind#15
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.

install-cli.sh rejects github.com as a non-canonical host under en_US.UTF-8 on macOS

1 participant