Author: scottbw
Date: Sun Nov  6 00:47:07 2011
New Revision: 1198117

URL: http://svn.apache.org/viewvc?rev=1198117&view=rev
Log:
Use Accept header to negotiate API content (see WOOKIE-286)

Modified:
    
incubator/wookie/trunk/src-tests/org/apache/wookie/tests/functional/WhiteListControllerTest.java
    
incubator/wookie/trunk/src-tests/org/apache/wookie/tests/functional/WidgetAccessRequestPolicyControllerTest.java
    incubator/wookie/trunk/src/org/apache/wookie/controller/Controller.java

Modified: 
incubator/wookie/trunk/src-tests/org/apache/wookie/tests/functional/WhiteListControllerTest.java
URL: 
http://svn.apache.org/viewvc/incubator/wookie/trunk/src-tests/org/apache/wookie/tests/functional/WhiteListControllerTest.java?rev=1198117&r1=1198116&r2=1198117&view=diff
==============================================================================
--- 
incubator/wookie/trunk/src-tests/org/apache/wookie/tests/functional/WhiteListControllerTest.java
 (original)
+++ 
incubator/wookie/trunk/src-tests/org/apache/wookie/tests/functional/WhiteListControllerTest.java
 Sun Nov  6 00:47:07 2011
@@ -119,7 +119,7 @@ public class WhiteListControllerTest ext
     HttpClient client = new HttpClient();
     GetMethod get = new GetMethod(WHITELIST_SERVICE_URL_VALID);
     setAuthenticationCredentials(client);
-    get.setRequestHeader("Content-type", "application/json");
+    get.setRequestHeader("Accept", "application/json");
     client.executeMethod(get);
     int code = get.getStatusCode();
     assertEquals(200, code);

Modified: 
incubator/wookie/trunk/src-tests/org/apache/wookie/tests/functional/WidgetAccessRequestPolicyControllerTest.java
URL: 
http://svn.apache.org/viewvc/incubator/wookie/trunk/src-tests/org/apache/wookie/tests/functional/WidgetAccessRequestPolicyControllerTest.java?rev=1198117&r1=1198116&r2=1198117&view=diff
==============================================================================
--- 
incubator/wookie/trunk/src-tests/org/apache/wookie/tests/functional/WidgetAccessRequestPolicyControllerTest.java
 (original)
+++ 
incubator/wookie/trunk/src-tests/org/apache/wookie/tests/functional/WidgetAccessRequestPolicyControllerTest.java
 Sun Nov  6 00:47:07 2011
@@ -231,7 +231,7 @@ public class WidgetAccessRequestPolicyCo
   public void testGetWithoutAuthentication() throws HttpException, IOException 
{
     HttpClient client = new HttpClient();
     GetMethod get = new GetMethod(TEST_WARP_SERVICE_URL_VALID);
-    get.addRequestHeader("content-type", "text/xml");
+    get.addRequestHeader("Accept", "text/xml");
     client.executeMethod(get);
     int code = get.getStatusCode();
     assertEquals(401, code);
@@ -251,7 +251,7 @@ public class WidgetAccessRequestPolicyCo
     setAuthenticationCredentials(client);
     GetMethod get = new GetMethod(TEST_WARP_SERVICE_URL_VALID);
     get.setDoAuthentication(true);
-    get.addRequestHeader("content-type", "text/xml");
+    get.addRequestHeader("Accept", "text/xml");
     client.executeMethod(get);
     int code = get.getStatusCode();
     assertEquals(200, code);

Modified: 
incubator/wookie/trunk/src/org/apache/wookie/controller/Controller.java
URL: 
http://svn.apache.org/viewvc/incubator/wookie/trunk/src/org/apache/wookie/controller/Controller.java?rev=1198117&r1=1198116&r2=1198117&view=diff
==============================================================================
--- incubator/wookie/trunk/src/org/apache/wookie/controller/Controller.java 
(original)
+++ incubator/wookie/trunk/src/org/apache/wookie/controller/Controller.java Sun 
Nov  6 00:47:07 2011
@@ -333,7 +333,7 @@ public abstract class Controller extends
         * request, this method will return HTML (1)
         */
        protected int format(HttpServletRequest request){
-               String type = request.getContentType();
+               String type = request.getHeader("Accept");
                if (type == null){
                        // check for format parameters in the request
                        if (request.getParameter("format")!=null){


Reply via email to