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

Fix invalid regex syntax #67

Merged
merged 1 commit into from
Oct 30, 2024
Merged

Fix invalid regex syntax #67

merged 1 commit into from
Oct 30, 2024

Conversation

felipecrs
Copy link

This fixes:

/app/classes/transports/modbus_base.py:108: SyntaxWarning: invalid escape sequence '\_'
  if not re.search("[^a-zA-Z0-9\_]", sn2) :
/app/classes/transports/modbus_base.py:265: SyntaxWarning: invalid escape sequence '\_'
  if val and not re.match('[^a-zA-Z0-9\_\-]', val): #validate ascii

Note these characters don't need to be escaped in such case.

@HotNoob
Copy link
Owner

HotNoob commented Oct 30, 2024

oops

@HotNoob HotNoob merged commit ae82b70 into HotNoob:v1.1.7 Oct 30, 2024
@felipecrs felipecrs deleted the v1.1.7 branch October 30, 2024 16:09
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.

2 participants