TheCoreDev/sterivein
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Apply firewall rules only in prod environment

Browse Source 
modifié:         core/lib/Thelia/Controller/BaseController.php
	modifié:         core/lib/Thelia/Form/FirewallForm.php
This commit is contained in:
Benjamin Perche
2014-08-12 15:07:37 +02:00
parent 9a43f5aa3c
commit ff16587acf
3 changed files with 37 additions and 33 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
core/lib/Thelia/Controller/BaseController.php
View File

@@ -201,7 +201,9 @@ abstract class BaseController extends ContainerAware
$form->bind($aBaseForm->getRequest()); $form->bind($aBaseForm->getRequest());
if ($form->isValid()) { if ($form->isValid()) {
if ($aBaseForm instanceof FirewallForm && !$aBaseForm->isFirewallOk()) { $env = $this->container->getParameter("kernel.environment");
if ($aBaseForm instanceof FirewallForm && !$aBaseForm->isFirewallOk($env)) {
throw new FormValidationException( throw new FormValidationException(
$this->getTranslator()->trans( $this->getTranslator()->trans(
"You've submitted this form too many times. Further submissions will be ignored during %time", "You've submitted this form too many times. Further submissions will be ignored during %time",

4
core/lib/Thelia/Form/FirewallForm.php
View File

@@ -32,8 +32,9 @@ abstract class FirewallForm extends BaseForm
const DEFAULT_TIME_TO_WAIT = 60; // 1 hour const DEFAULT_TIME_TO_WAIT = 60; // 1 hour
const DEFAULT_ATTEMPTS = 6; const DEFAULT_ATTEMPTS = 6;
public function isFirewallOk() public function isFirewallOk($env)
{ {
if ($env === "prod") {
/** /**
* Empty the firewall * Empty the firewall
*/ */
@@ -68,6 +69,7 @@ abstract class FirewallForm extends BaseForm
$firewallInstance->save(); $firewallInstance->save();
} }
}
return true; return true;
} }

10
core/lib/Thelia/Tests/Form/FirewallTest.php
View File

@@ -88,11 +88,11 @@ class FirewallTest extends \PHPUnit_Framework_TestCase
if ($i > 6) { if ($i > 6) {
$this->assertFalse( $this->assertFalse(
$this->form->isFirewallOk() $this->form->isFirewallOk("prod")
); );
} else { } else {
$this->assertTrue( $this->assertTrue(
$this->form->isFirewallOk() $this->form->isFirewallOk("prod")
); );
} }
@@ -145,7 +145,7 @@ class FirewallTest extends \PHPUnit_Framework_TestCase
public function testAutoDelete() public function testAutoDelete()
{ {
/** Add two rows */ /** Add two rows */
$this->form->isFirewallOk(); $this->form->isFirewallOk("prod");
$this->form $this->form
->expects($this->any()) ->expects($this->any())
@@ -153,7 +153,7 @@ class FirewallTest extends \PHPUnit_Framework_TestCase
->will($this->returnValue("test_form_firewall_2")) ->will($this->returnValue("test_form_firewall_2"))
; ;
$this->form->isFirewallOk(); $this->form->isFirewallOk("prod");
/** Set the time to 1h and 1s after the limit */ /** Set the time to 1h and 1s after the limit */
FormFirewallQuery::create() FormFirewallQuery::create()
@@ -162,7 +162,7 @@ class FirewallTest extends \PHPUnit_Framework_TestCase
->save() ->save()
; ;
$this->form->isFirewallOk(); $this->form->isFirewallOk("prod");
/** Assert that the table is empty */ /** Assert that the table is empty */
$this->assertEquals( $this->assertEquals(