From e3b7330de210f6da2f5585398ab4544a3ad0f2d7 Mon Sep 17 00:00:00 2001
From: Jerome Berthier <jerome.berthier@budget-insight.com>
Date: Fri, 3 May 2019 14:29:30 +0200
Subject: [PATCH] [banquepopulaire] Fix error 400 during login

It seems this field 'id' was never verified by the server.
Now when credentials are invalid it is checked and server
returns 400 if it is invalid.

Behavior seen on regions Occitanie, PACA and Alsace
---
 modules/banquepopulaire/pages.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/modules/banquepopulaire/pages.py b/modules/banquepopulaire/pages.py
index 48f6811496..f0eec83248 100644
--- a/modules/banquepopulaire/pages.py
+++ b/modules/banquepopulaire/pages.py
@@ -396,7 +396,7 @@ def login(self, login, password):
             payload = {
                 'validate': {
                     form_id[0]: [{
-                        'id': self.form_id[1],
+                        'id': form_id[1],
                         'password': password,
                         'type': 'PASSWORD',
                     }]
@@ -404,7 +404,6 @@ def login(self, login, password):
             }
         r = self.browser.open(url, json=payload)
 
-
         doc = r.json()
         self.logger.debug('doc = %s', doc)
         if 'phase' in doc and doc['phase']['state'] == 'TERMS_OF_USE':
-- 
GitLab