Revision
9313
Author
cdaboo@apple.com
Date
2012-05-30 10:23:34 -0700 (Wed, 30 May 2012)

Log Message

Auth types are not case-sensitive.

Modified Paths

Added Paths

Diff

Modified: CalendarServer/trunk/contrib/performance/httpauth.py (9312 => 9313)


--- CalendarServer/trunk/contrib/performance/httpauth.py	2012-05-30 17:23:01 UTC (rev 9312)
+++ CalendarServer/trunk/contrib/performance/httpauth.py	2012-05-30 17:23:34 UTC (rev 9313)
@@ -104,7 +104,7 @@
         challengeType = {
             'basic': BasicChallenge,
             'digest': DigestChallenge,
-            }.get(scheme)
+            }.get(scheme.lower())
         if challengeType is None:
             return None
         return challengeType(**args)

Added: CalendarServer/trunk/contrib/performance/test_httpauth.py (0 => 9313)


--- CalendarServer/trunk/contrib/performance/test_httpauth.py	                        (rev 0)
+++ CalendarServer/trunk/contrib/performance/test_httpauth.py	2012-05-30 17:23:34 UTC (rev 9313)
@@ -0,0 +1,40 @@
+##
+# Copyright (c) 2012 Apple Inc. All rights reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+##
+
+from twisted.trial.unittest import TestCase
+from httpauth import AuthHandlerAgent
+
+class HTTPAuthTests(TestCase):
+
+
+    def test_AuthHandlerAgent_parse(self):
+        """
+        Make sure auth method is handled case-insensitively.
+        """
+        headers = (
+            "Basic realm=\"foo\"",
+            "basic realm=\"foo\"",
+            "BASIC realm=\"foo\"",
+            "Digest realm=\"foo\"",
+            "digest realm=\"foo\"",
+            "DIGEST realm=\"foo\"",
+        )
+
+        for hdrvalue in headers:
+            
+            hdlr = AuthHandlerAgent(None, None)
+            ch = hdlr._parse(hdrvalue)
+            self.assertTrue(ch is not None)