Changeset 10411 for main/waeup.cas

Timestamp:

6 Jul 2013, 09:58:16 (11 years ago)

Author:

uli

Message:

Add support for renew parameter.

Location:

main/waeup.cas/trunk/waeup/cas

Files:

• server.py (modified) (2 diffs)
• tests/test_server.py (modified) (1 diff)

main/waeup.cas/trunk/waeup/cas/server.py

@@ -208 +208 @@
     def login(self, req):
         service = req.POST.get('service', req.GET.get('service', None))
+        renew = req.POST.get('renew', req.GET.get('renew', None))
         service_field = ''
         msg = ''
@@ -214 +215 @@
         valid_lt = check_login_ticket(self.db, req.POST.get('lt'))
         tgc = check_session_cookie(self.db, req.cookies.get('cas-tgc', None))
-        if tgc:
+        if tgc and renew is None:
             if service:
                 return login_redirect_service(self.db, service, sso=True)

main/waeup.cas/trunk/waeup/cas/tests/test_server.py

@@ -126 +126 @@
         assert 'Set-Cookie' not in resp.headers
         return
+
+    def test_login_renew_with_cookie(self):
+        # 2.1.1: cookie will be ignored when renew is set
+        app = CASServer()
+        tgc = create_tgc_value()
+        app.db.add(tgc)
+        value = str(tgc.value)
+        req = Request.blank('https://localhost/login?renew=true')
+        req.headers['Cookie'] = 'cas-tgc=%s' % value
+        resp = app(req)
+        assert resp.status == '200 OK'
+        assert b'username' in resp.body
+        assert 'Set-Cookie' not in resp.headers
+
+    def test_login_renew_as_empty_string(self):
+        # `renew` is handled correctly, even with empty value
+        app = CASServer()
+        tgc = create_tgc_value()
+        app.db.add(tgc)
+        value = str(tgc.value)
+        req = Request.blank('https://localhost/login?renew')
+        req.headers['Cookie'] = 'cas-tgc=%s' % value
+        resp = app(req)
+        assert resp.status == '200 OK'
+        assert b'username' in resp.body
+        assert 'Set-Cookie' not in resp.headers