From d699c9ace8028bf74e1c6c3ea085fdf9beeb2586 Mon Sep 17 00:00:00 2001
From: Theron Luhn <theron@luhn.com>
Date: Sat, 14 Dec 2019 19:12:01 -0600
Subject: Raise error on kwargs in `LegacySecurityPolicy.forget`.

---
 src/pyramid/security.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/pyramid/security.py b/src/pyramid/security.py
index f07bc0bfe..e3a978c52 100644
--- a/src/pyramid/security.py
+++ b/src/pyramid/security.py
@@ -439,8 +439,12 @@ class LegacySecurityPolicy:
         return authn.remember(request, userid, **kw)
 
     def forget(self, request, **kw):
+        if kw:
+            raise ValueError(
+                'Legacy authentication policies do not support keyword '
+                'arguments for `forget`'
+            )
         authn = self._get_authn_policy(request)
-        # XXX log warning if varkwargs were passed?
         return authn.forget(request)
 
     def permits(self, request, context, permission):
-- 
cgit v1.2.3