Merge pull request #383 from mozilla-services/bug/normalize_headers

bug: force header values to lowercase underscored values
mozilla-services · Mar 4, 2016 · 86e0ab1 · 86e0ab1
2 parents 57243be + b4517ae
commit 86e0ab1
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/autopush/router/webpush.py b/autopush/router/webpush.py
@@ -54,8 +54,9 @@ def _crypto_headers(self, notification):
         # they're present to avoid empty strings.
         for name in ["encryption-key", "crypto-key"]:
             if name in headers:
-                # Normalize hyphens in header names.
-                data[name.replace("-", "_")] = headers[name]
+                # NOTE: The client code expects all header keys to be lower
+                # case and s/-/_/.
+                data[name.lower().replace("-", "_")] = headers[name]
         return data
 
     @inlineCallbacks