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

Get Started page does not scrub/strip worker name field #608

Open
indyramp opened this issue Feb 14, 2022 · 0 comments
Open

Get Started page does not scrub/strip worker name field #608

indyramp opened this issue Feb 14, 2022 · 0 comments
Assignees
Labels
priority:moderate Issue is under moderate priority rel:ux Related to UX type:bug This is a bug

Comments

@indyramp
Copy link

The "worker name" field in the Get Started command builder does not check for valid worker names. This means spaces, punctuation, and potentially other things could get in there.

The field should scrub anything that is not a letter, number, underscore, or hyphen.

In the event that any other characters are valid, the miner should be able to manually adjust, but we should not allow spaces, emojis, shell characters (for linux miners), or other invalid characters.

image

image

@indyramp indyramp added priority:moderate Issue is under moderate priority rel:ux Related to UX type:bug This is a bug labels Feb 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:moderate Issue is under moderate priority rel:ux Related to UX type:bug This is a bug
Projects
None yet
Development

No branches or pull requests

2 participants