Add Server field in HTTP API

Resolves #625 Included in tests. License: MIT Signed-off-by: Jakub (Kubuxu) Sztandera <kubuxu@gmail.com>
ipfs · Jan 12, 2016 · 1d6e577 · 1d6e577
1 parent 04914ac
commit 1d6e577
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/commands/http/handler.go b/commands/http/handler.go
@@ -199,6 +199,10 @@ func guessMimeType(res cmds.Response) (string, error) {
 }
 
 func sendResponse(w http.ResponseWriter, r *http.Request, res cmds.Response, req cmds.Request) {
+ h := w.Header()
+ // Expose our agent to allow identification
+ h.Set("Server", "go-ipfs/" + config.CurrentVersionNumber)
+
  mime, err := guessMimeType(res)
  if err != nil {
  http.Error(w, err.Error(), http.StatusInternalServerError)
@@ -222,8 +226,6 @@ func sendResponse(w http.ResponseWriter, r *http.Request, res cmds.Response, req
  return
  }
 
- h := w.Header()
-
  // Set up our potential trailer
  h.Set("Trailer", StreamErrHeader)
 

diff --git a/test/sharness/t0230-channel-streaming-http-content-type.sh b/test/sharness/t0230-channel-streaming-http-content-type.sh
@@ -24,6 +24,7 @@ test_ls_cmd() {
  printf "Access-Control-Allow-Headers: X-Stream-Output, X-Chunked-Output\r\n" >>expected_output &&
  printf "Access-Control-Expose-Headers: X-Stream-Output, X-Chunked-Output\r\n" >>expected_output &&
  printf "Content-Type: text/plain\r\n" >>expected_output &&
+ printf "Server: go-ipfs/%s\r\n" $(ipfs version -n) >>expected_output &&
  printf "Trailer: X-Stream-Error\r\n" >>expected_output &&
  printf "Transfer-Encoding: chunked\r\n" >>expected_output &&
  printf "X-Chunked-Output: 1\r\n" >>expected_output &&
@@ -46,6 +47,7 @@ test_ls_cmd() {
  printf "Access-Control-Allow-Headers: X-Stream-Output, X-Chunked-Output\r\n" >>expected_output &&
  printf "Access-Control-Expose-Headers: X-Stream-Output, X-Chunked-Output\r\n" >>expected_output &&
  printf "Content-Type: application/json\r\n" >>expected_output &&
+ printf "Server: go-ipfs/%s\r\n" $(ipfs version -n) >>expected_output &&
  printf "Trailer: X-Stream-Error\r\n" >>expected_output &&
  printf "Transfer-Encoding: chunked\r\n" >>expected_output &&
  printf "X-Chunked-Output: 1\r\n" >>expected_output &&