-
Notifications
You must be signed in to change notification settings - Fork 46
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
[Bug]: Found a few more regex issues and a python package issue #40
Labels
Comments
gtjoseph
added a commit
to gtjoseph/testsuite
that referenced
this issue
Feb 16, 2024
Fixed a few more regex escaping issues in config.py and in opensslversion.py. Replaced cryptography with pyopenssl in requirements.txt because cryptography doesn't include the OpenSSL module which we use to get the openssl version. Tests that had "openssl" as a prereq have not been running because of this. Resolves: asterisk#40
asterisk-org-access-app bot
pushed a commit
that referenced
this issue
Feb 27, 2024
Fixed a few more regex escaping issues in config.py and in opensslversion.py. Replaced cryptography with pyopenssl in requirements.txt because cryptography doesn't include the OpenSSL module which we use to get the openssl version. Tests that had "openssl" as a prereq have not been running because of this. Resolves: #40
asterisk-org-access-app bot
pushed a commit
that referenced
this issue
Feb 27, 2024
Fixed a few more regex escaping issues in config.py and in opensslversion.py. Replaced cryptography with pyopenssl in requirements.txt because cryptography doesn't include the OpenSSL module which we use to get the openssl version. Tests that had "openssl" as a prereq have not been running because of this. Resolves: #40
asterisk-org-access-app bot
pushed a commit
that referenced
this issue
Feb 27, 2024
Fixed a few more regex escaping issues in config.py and in opensslversion.py. Replaced cryptography with pyopenssl in requirements.txt because cryptography doesn't include the OpenSSL module which we use to get the openssl version. Tests that had "openssl" as a prereq have not been running because of this. Resolves: #40
asterisk-org-access-app bot
pushed a commit
that referenced
this issue
Feb 27, 2024
Fixed a few more regex escaping issues in config.py and in opensslversion.py. Replaced cryptography with pyopenssl in requirements.txt because cryptography doesn't include the OpenSSL module which we use to get the openssl version. Tests that had "openssl" as a prereq have not been running because of this. Resolves: #40
asterisk-org-access-app bot
pushed a commit
that referenced
this issue
Feb 27, 2024
Fixed a few more regex escaping issues in config.py and in opensslversion.py. Replaced cryptography with pyopenssl in requirements.txt because cryptography doesn't include the OpenSSL module which we use to get the openssl version. Tests that had "openssl" as a prereq have not been running because of this. Resolves: #40
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Severity
Minor
Versions
18,20,21,master
Components/Modules
All
Operating Environment
All
Frequency of Occurrence
Constant
Issue Description
There are a few more regex escaping issues in config.py and one in opensslversion.py.
There's also an issue in requirements.txt. We're installing the cryptography package to get openssl features but that package doesn't include the actual OpenSSL module whic is what we use to determine openssl version. The result is that any test that has a prerequisite on "openssl" hasn't been running. We need to install the pyOpenSSL and service_identity packages instead.
Relevant log output
No response
Asterisk Issue Guidelines
The text was updated successfully, but these errors were encountered: