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

fixes #35

Closed
wants to merge 76 commits into from
Closed

fixes #35

wants to merge 76 commits into from

Conversation

CyrilP
Copy link
Owner

@CyrilP CyrilP commented Nov 9, 2023

  • fix local connection
  • fix config flow issue
  • fix manifest

disallowed = re.compile(r"[^a-zA-Z\d\-]")
return all(x and not disallowed.search(x) for x in host.split("."))

regex = re.compile(r"([A-Za-z0-9]+[.-_])*[A-Za-z0-9]+@[A-Za-z0-9-]+(\.[A-Z|a-z]{2,})+")

Check warning

Code scanning / CodeQL

Overly permissive regular expression range Medium

Suspicious character range that is equivalent to \[.\/0-9:;<=>?@A-Z\\[\\\\]^_\].
@CyrilP CyrilP closed this Nov 11, 2023
@CyrilP CyrilP deleted the new branch November 11, 2023 12:47
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.

1 participant