Skip to content
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.

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

SSH key regular expression is too restrictive #40

Open
scop opened this issue Nov 13, 2024 · 1 comment
Open

SSH key regular expression is too restrictive #40

scop opened this issue Nov 13, 2024 · 1 comment

Comments

@scop
Copy link
Member

scop commented Nov 13, 2024

The regular expression used to validate SSH public key to use is too restrictive, it allows RSA keys only, and disallows ed25519 ones among others.

['fieldname' => 'ssh_key|SSH Public Key', 'fieldtype' => 'textarea', 'description' => $this->_LANG["sshRsa"], 'regexpr' => "#ssh-rsa AAAA[0-9A-Za-z+/]+[=]{0,3}( [^@]+@[^@]+)?#", "showorder" => "on"],

I wonder if this validation would be better off removed altogether, and have the UpCloud API do its validation on it? I'd say this would be the way to go if it yields good enough UX on errors.

@adityawbwce
Copy link
Contributor

This will be replaced if we are addressing issue #44

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

No branches or pull requests

3 participants