Skip to content

Commit e84dfee

Browse files
committed
Update hCaptcha integration to use the new submit signature without remoteip, aligning with plone.formwidget.hcaptcha privacy (GDPR/LGPD) change
1 parent b2729cc commit e84dfee

File tree

4 files changed

+5
-7
lines changed

4 files changed

+5
-7
lines changed

news/88.bugfix

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Update hCaptcha integration to use the new submit signature without remoteip, aligning with plone.formwidget.hcaptcha privacy (GDPR/LGPD) change. @alexandreIFB

setup.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@
7373
],
7474
extras_require={
7575
"hcaptcha": [
76-
"plone.formwidget.hcaptcha>=1.0.1",
76+
"plone.formwidget.hcaptcha>=1.0.3",
7777
],
7878
"recaptcha": [
7979
"plone.formwidget.recaptcha",

src/collective/volto/formsupport/captcha/hcaptcha.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,10 +46,7 @@ def verify(self, data):
4646
)
4747
)
4848
token = data["token"]
49-
remote_addr = self.request.get("HTTP_X_FORWARDED_FOR", "").split(",")[0]
50-
if not remote_addr:
51-
remote_addr = self.request.get("REMOTE_ADDR")
52-
res = submit(token, self.settings.private_key, remote_addr)
49+
res = submit(token, self.settings.private_key)
5350
if not res.is_valid:
5451
raise BadRequest(
5552
translate(

src/collective/volto/formsupport/tests/test_captcha.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@ def test_hcaptcha(
276276
},
277277
)
278278
transaction.commit()
279-
mock_submit.assert_called_once_with("12345", "private", "127.0.0.1")
279+
mock_submit.assert_called_once_with("12345", "private")
280280
self.assertEqual(response.status_code, 400)
281281
self.assertEqual(
282282
response.json()["message"],
@@ -301,7 +301,7 @@ def test_hcaptcha(
301301
},
302302
)
303303
transaction.commit()
304-
mock_submit.assert_called_once_with("12345", "private", "127.0.0.1")
304+
mock_submit.assert_called_once_with("12345", "private")
305305
self.assertEqual(response.status_code, 200)
306306

307307
def test_get_vocabulary(self):

0 commit comments

Comments
 (0)