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

Add transport to asterisk RE #2913

Merged
merged 3 commits into from Jan 31, 2021
Merged

Add transport to asterisk RE #2913

merged 3 commits into from Jan 31, 2021

Conversation

brianjmurrell
Copy link
Contributor

Call rejection messages from Asterisk can have the transport prefixed to the IP address.

Signed-off-by: Brian J. Murrell brian@interlinx.bc.ca

Call rejection messages from Asterisk can have the transport prefixed to the IP address.

Signed-off-by: Brian J. Murrell <brian@interlinx.bc.ca>
Add a sample failregex.

Signed-off-by: Brian J. Murrell <brian@interlinx.bc.ca>
@brianjmurrell
Copy link
Contributor Author

CI tests seem to fail with:

ValueError: time data 'Jan 18 17:39:50' does not match format '%Y-%m-%dT%H:%M:%S.%f'
540

But the date format I put in the example line is that of my Asterisk server. Am I supposed to contort that into the date format of the fail2ban/tests/files/logs/asterisk file, or is it a bug that it's not recognizing the date format of my Asterisk logging?

The change as it is here is working in my local installation, using the date format from my Asterisk log.

The date format in failJSON is specific, so convert the date to use that format.
@brianjmurrell
Copy link
Contributor Author

Can this be landed and closed?

@sebres sebres merged commit 7f185a8 into fail2ban:master Jan 31, 2021
@sebres
Copy link
Contributor

sebres commented Jan 31, 2021

Sure. Done.

sebres added a commit that referenced this pull request Feb 3, 2021
Add transport to asterisk RE
@brianjmurrell brianjmurrell deleted the patch-3 branch July 17, 2021 14:14
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.

None yet

2 participants