On 6/28/06, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:

@@ -82,6 +83,18 @@
       if (context.getLocation() != null)
         response.setHeader("Location", context.getLocation().toString());
       handleCachePolicy(response, context.getCachePolicy());
+
+      // Add any custom headers after we've set the known ones,
+      // giving the developer an option to replace or set multiple
+      // headers. If they want to skip the ones above, they simply
+      // don't set them.
+      Map<String, String> headers = context.getHeaders();
+      if(headers != null) {
+        for(String header : headers.keySet()) {
+          response.setHeader(header, headers.get(header));
+        }
+      }

The spacing here (between if/for and the following paren) is off
compared to the rest of this file.  Keeping that sort of thing
consistent helps with readability.

-garrett

Reply via email to