New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Input validation #464

Open

lentalamp wants to merge 3 commits into killercup:master from lentalamp:input-validation

lentalamp commented May 22, 2021

Add some checks when parsing crate names passed as arguments:

Check whether the name starts with a letter and contains only letters, numbers, dashes and hyphens (same requirements as crates.io has).
Disallow passing '.' as path to crate to prevent accidental creation of circular dependencies.

na added 3 commits

May 22, 2021 13:52


          Adding validate_name methon on crate_name

bb99dc4


          Validating input by checking crate names.

3ebad16

Added error CrateNameContainsInvalidCharacter.
Blacklisted crate name '.'


          Crate name validation -check if 1st char is letter

41fe392

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet