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

Closes #2751 filter.d/sendmail-*.local configparser recursion issue #2752

Closed
wants to merge 2 commits into from

Conversation

yolabingo
Copy link

Before submitting your PR, please review the following checklist:

  • CHOOSE CORRECT BRANCH: if filing a bugfix/enhancement
    against certain release version, choose 0.9, 0.10 or 0.11 branch,
    for dev-edition use master branch
  • CONSIDER adding a unit test if your PR resolves an issue
  • LIST ISSUES this PR resolves
  • MAKE SURE this PR doesn't break existing tests
  • KEEP PR small so it could be easily reviewed.
  • AVOID making unnecessary stylistic changes in unrelated code
  • ACCOMPANY each new failregex for filter X with sample log lines
    within fail2ban/tests/files/logs/X file

@codecov-commenter
Copy link

codecov-commenter commented Jun 6, 2020

Codecov Report

Merging #2752 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2752      +/-   ##
==========================================
+ Coverage   95.84%   95.86%   +0.01%     
==========================================
  Files          78       78              
  Lines       15750    15750              
  Branches     2560     2560              
==========================================
+ Hits        15096    15098       +2     
+ Misses        350      348       -2     
  Partials      304      304              
Impacted Files Coverage Δ
fail2ban/server/utils.py 90.07% <0.00%> (+0.39%) ⬆️
fail2ban/server/filterpyinotify.py 94.31% <0.00%> (+0.47%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6cff2bb...3c03514. Read the comment docs.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 98.038% when pulling 3c03514 on yolabingo:master into 6cff2bb on fail2ban:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 98.038% when pulling 3c03514 on yolabingo:master into 6cff2bb on fail2ban:master.

@sebres
Copy link
Contributor

sebres commented Aug 26, 2020

See #2751 (comment)

@sebres sebres closed this Aug 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants