nickva commented on a change in pull request #3568:
URL: https://github.com/apache/couchdb/pull/3568#discussion_r633806178



##########
File path: src/chttpd/src/chttpd.erl
##########
@@ -537,15 +641,22 @@ extract_cookie(#httpd{mochi_req = MochiReq}) ->
 %%% end hack
 
 set_auth_handlers() ->
-    AuthenticationDefault =  "{chttpd_auth, cookie_authentication_handler},
-      {chttpd_auth, default_authentication_handler}",
+    AuthenticationDefault =
+        "{chttpd_auth, cookie_authentication_handler},\n"

Review comment:
       0
   
   I think it auto-inserted a `\n` character in there and split it into two 
lines. The result is the same  but would need a bit extra scrutiny since it is 
the authentication handler of all places. We may want by bypass these sections 
even on the first go around.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to