support JSONP in externals

COUCHDB-1313
rcouch · Feb 9, 2013 · 52444eb · 52444eb
1 parent a2b2828
commit 52444eb
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 8 deletions.
diff --git a/apps/couch/include/couch_db.hrl b/apps/couch/include/couch_db.hrl
@@ -207,7 +207,8 @@
     stop = false,
     data = <<>>,
     ctype = "application/json",
-    headers = []
+    headers = [],
+    json = nil
 }).
 
 -record(index_header,

diff --git a/apps/couch_httpd/src/couch_httpd_external.erl b/apps/couch_httpd/src/couch_httpd_external.erl
@@ -122,17 +122,21 @@ json_query_keys([{<<"key">>, Value} | Rest], Acc) ->
 json_query_keys([Term | Rest], Acc) ->
     json_query_keys(Rest, [Term|Acc]).
 
-send_external_response(#httpd{mochi_req=MochiReq}=Req, Response) ->
+send_external_response(Req, Response) ->
     #extern_resp_args{
         code = Code,
         data = Data,
         ctype = CType,
-        headers = Headers
+        headers = Headers,
+        json = Json
     } = parse_external_response(Response),
-    couch_httpd:log_request(Req, Code),
+    Headers1 = default_or_content_type(CType, Headers),
-    Resp = MochiReq:respond({Code,
+    case Json of
-        default_or_content_type(CType, Headers ++ couch_httpd:server_header()), Data}),
+    nil ->
-    {ok, Resp}.
+        couch_httpd:send_response(Req, Code, Headers1, Data);
+    Json ->
+        couch_httpd:send_json(Req, Code, Headers1, Json)
+    end.
 
 parse_external_response({Response}) ->
     lists:foldl(fun({Key,Value}, Args) ->
@@ -145,7 +149,7 @@ parse_external_response({Response}) ->
                 Args#extern_resp_args{stop=true};
             {<<"json">>, Value} ->
                 Args#extern_resp_args{
-                    data=?JSON_ENCODE(Value),
+                    json=Value,
                     ctype="application/json"};
             {<<"body">>, Value} ->
                 Args#extern_resp_args{data=Value, ctype="text/html; charset=utf-8"};