Skip to content

Commit

Permalink
Merge 40eb80b into 435dfce
Browse files Browse the repository at this point in the history
  • Loading branch information
ryandeivert committed Jul 7, 2020
2 parents 435dfce + 40eb80b commit 5caa48a
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions streamalert_cli/apps/handler.py
Expand Up @@ -142,9 +142,9 @@ def _add_default_app_args(app_parser):
# Validate the name being used to make sure it does not contain specific characters
def _validate_name(val):
"""Validate acceptable inputs for the name of the function"""
acceptable_chars = ''.join([string.digits, string.ascii_letters, '_-'])
acceptable_chars = ''.join([string.digits, string.ascii_lowercase, '_-'])
if not set(str(val)).issubset(acceptable_chars):
raise app_parser.error('Name must contain only letters, numbers, '
raise app_parser.error('Name must contain only lowercase letters, numbers, '
'hyphens, or underscores.')

return val
Expand Down
4 changes: 2 additions & 2 deletions streamalert_cli/config.py
Expand Up @@ -97,9 +97,9 @@ def set_prefix(self, prefix):
LOGGER.error('Invalid prefix type, must be string')
return False

acceptable_chars = set([*string.digits, *string.ascii_letters])
acceptable_chars = set([*string.digits, *string.ascii_lowercase])
if not set(prefix).issubset(acceptable_chars):
LOGGER.error('Prefix must contain only letters and numbers')
LOGGER.error('Prefix must contain only lowercase letters and numbers')
return False

self.config['global']['account']['prefix'] = prefix
Expand Down

0 comments on commit 5caa48a

Please sign in to comment.