Changeset 107

Timestamp:

08/09/07 13:08:54

Author:

thejimmyg

Message:

Changed form middleware so you can choose which status it returns and so that it tells the Pylons error docuement middleware not to intercept a 401 response it generates

Files:

• AuthKit/branches/0.4/authkit/authenticate/form.py (modified) (4 diffs)
• AuthKit/branches/0.4/authkit/authenticate/multi.py (modified) (1 diff)

AuthKit/branches/0.4/authkit/authenticate/form.py

@@ -4 +4 @@
-detail in the AuthKit manual and should be used via the
-``authkit.authenticate.middleware`` function.
+
+The option form.status can be set to "200 OK" if the Pylons error document
+middleware is intercepting the 401 response and just showing the standard 401
+error document. This will not happen in recent versions of Pylons (0.9.6)
+because this middleware sets the environ['pylons.error_call'] key so that the
+error documents middleware doesn't intercept the response.
-"""
-
@@ -11 +17 @@
-   get_authenticate_function, strip_base, RequireEnvironKey
-from authkit.authenticate.multi import MultiHandler, status_checker
+
+import logging
+log = logging.getLogger('authkit.authenticate.form')
-
-def template():
@@ -36 +45 @@
-        app,
-        charset=None,
+        status="401 Unauthorized",
-        **p
-    ):
-        AuthFormHandler.__init__(self, app, **p)
+        self.status = status
-        if charset is None:
-            self.charset = ''
@@ -65 +76 @@
-                    del environ['paste.parsed_formvars']
-                    return self.on_authorized(environ, start_response)
-
-
-
+
+
+
+
+
+
+
-                                 ('Content-Length', str(len(content)))])
-        return [content]

AuthKit/branches/0.4/authkit/authenticate/multi.py

@@ -53 +53 @@
-                if checker(environ, status_[0], headers_ and headers_[0] or []):
-                    log.debug(
-
+i
-                        binding
-                    )