Skip to content

Firewall improvement#14450

Merged
niden merged 14 commits into
phalcon:4.0.xfrom
ruudboon:firewall-improvement
Oct 5, 2019
Merged

Firewall improvement#14450
niden merged 14 commits into
phalcon:4.0.xfrom
ruudboon:firewall-improvement

Conversation

@ruudboon

@ruudboon ruudboon commented Oct 5, 2019

Copy link
Copy Markdown
Member

Hello!

In raising this pull request, I confirm the following:

  • I have read and understood the Contributing Guidelines
  • I have checked that another pull request for this purpose does not exist
  • I wrote some tests for this PR
  • I have updated the relevant CHANGELOG
  • I have created a PR for the documentation about this change

Small description of change:

Thanks

Comment thread phalcon/Firewall/Adapter/AbstractAdapter.zep Outdated
@codecov

codecov Bot commented Oct 5, 2019

Copy link
Copy Markdown

Codecov Report

Merging #14450 into 4.0.x will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            4.0.x   #14450      +/-   ##
==========================================
- Coverage   67.71%    67.7%   -0.01%     
==========================================
  Files         482      482              
  Lines      111207   111222      +15     
==========================================
+ Hits        75304    75308       +4     
- Misses      35903    35914      +11

@niden niden added 4.0 documentation Documentation required labels Oct 5, 2019
sergeyklay
sergeyklay previously approved these changes Oct 5, 2019
@niden niden merged commit cfe2c02 into phalcon:4.0.x Oct 5, 2019
@niden niden added bug A bug report status: low Low and removed Bug - Low labels Dec 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug A bug report documentation Documentation required status: low Low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants