Merge pull request #1312 from wallabag/v2-fix-1311

fix #1311, redirect after login

2 files changed, 21 insertions, 2 deletions

diff --git a/app/config/security.yml b/app/config/security.yml
index 37236d40..acc86984 100644
--- a/app/config/security.yml
+++ b/app/config/security.yml
@@ -37,9 +37,9 @@ security:
 
                 post_only:                      true
 
-                always_use_default_target_path: true
+                always_use_default_target_path: false
                 default_target_path:            /
-                target_path_parameter:          redirect_url
+                target_path_parameter:          _target_path
                 use_referer:                    true
 
                 failure_path:                   null
diff --git a/src/Wallabag/CoreBundle/Tests/Controller/SecurityControllerTest.php b/src/Wallabag/CoreBundle/Tests/Controller/SecurityControllerTest.php
index a51e836d..759ef01b 100644
--- a/src/Wallabag/CoreBundle/Tests/Controller/SecurityControllerTest.php
+++ b/src/Wallabag/CoreBundle/Tests/Controller/SecurityControllerTest.php
@@ -40,6 +40,25 @@ class SecurityControllerTest extends WallabagCoreTestCase
         $this->assertContains('Bad credentials', $client->getResponse()->getContent());
     }
 
+    public function testRedirectionAfterLogin()
+    {
+        $client = $this->getClient();
+        $client->followRedirects();
+
+        $crawler = $client->request('GET', '/config');
+
+        $form = $crawler->filter('button[type=submit]')->form();
+
+        $data = array(
+            '_username' => 'admin',
+            '_password' => 'mypassword',
+        );
+
+        $client->submit($form, $data);
+
+        $this->assertContains('RSS', $client->getResponse()->getContent());
+    }
+
     public function testForgotPassword()
     {
         $client = $this->getClient();