Update src/Symfony/Component/Security/Http/Firewall/UsernamePasswordFormAuthenticationListener.php

nervo · nervo · commit e0c001b772dd · 2012-09-23T10:54:29.000+03:00
Better consistency in request method checking (See L.58)
diff --git a/src/Symfony/Component/Security/Http/Firewall/UsernamePasswordFormAuthenticationListener.php b/src/Symfony/Component/Security/Http/Firewall/UsernamePasswordFormAuthenticationListener.php
@@ -67,7 +67,7 @@ protected function requiresAuthentication(Request $request)
      */
     protected function attemptAuthentication(Request $request)
     {
-        if ($this->options['post_only'] && 'post' !== strtolower($request->getMethod())) {
+        if ($this->options['post_only'] && !$request->isMethod('post')) {
             if (null !== $this->logger) {
                 $this->logger->debug(sprintf('Authentication method not supported: %s.', $request->getMethod()));
             }

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ protected function requiresAuthentication(Request $request)`
`67`	`67`	`*/`
`68`	`68`	`protected function attemptAuthentication(Request $request)`
`69`	`69`	`{`
`70`		`- if ($this->options['post_only'] && 'post' !== strtolower($request->getMethod())) {`
	`70`	`+ if ($this->options['post_only'] && !$request->isMethod('post')) {`
`71`	`71`	`if (null !== $this->logger) {`
`72`	`72`	`$this->logger->debug(sprintf('Authentication method not supported: %s.', $request->getMethod()));`
`73`	`73`	`}`