docs(api): document download params honored by HEAD

Author: gacevicljubisa

openapi/Swarm.yaml

@@ -255,12 +255,17 @@ paths:
             $ref: "SwarmCommon.yaml#/components/schemas/SwarmAddress"
           required: true
           description: Swarm address of chunk
+        - $ref: "SwarmCommon.yaml#/components/parameters/SwarmCache"
+        - $ref: "SwarmCommon.yaml#/components/parameters/SwarmRedundancyStrategyParameter"
+        - $ref: "SwarmCommon.yaml#/components/parameters/SwarmRedundancyFallbackModeParameter"
+        - $ref: "SwarmCommon.yaml#/components/parameters/SwarmRedundancyLevelParameter"
+        - $ref: "SwarmCommon.yaml#/components/parameters/SwarmChunkRetrievalTimeoutParameter"
         - $ref: "SwarmCommon.yaml#/components/parameters/SwarmActTimestamp"
         - $ref: "SwarmCommon.yaml#/components/parameters/SwarmActPublisher"
         - $ref: "SwarmCommon.yaml#/components/parameters/SwarmActHistoryAddress"
       responses:
         "200":
-          description: The chunk exists.
+          description: Headers for the content at the reference.
           headers:
             Content-Type:
               description: The MIME type of the resource (e.g., application/octet-stream).
@@ -489,12 +494,17 @@ paths:
             $ref: "SwarmCommon.yaml#/components/schemas/SwarmAddress"
           required: true
           description: Swarm address of chunk
+        - $ref: "SwarmCommon.yaml#/components/parameters/SwarmCache"
+        - $ref: "SwarmCommon.yaml#/components/parameters/SwarmRedundancyStrategyParameter"
+        - $ref: "SwarmCommon.yaml#/components/parameters/SwarmRedundancyFallbackModeParameter"
+        - $ref: "SwarmCommon.yaml#/components/parameters/SwarmRedundancyLevelParameter"
+        - $ref: "SwarmCommon.yaml#/components/parameters/SwarmChunkRetrievalTimeoutParameter"
         - $ref: "SwarmCommon.yaml#/components/parameters/SwarmActTimestamp"
         - $ref: "SwarmCommon.yaml#/components/parameters/SwarmActPublisher"
         - $ref: "SwarmCommon.yaml#/components/parameters/SwarmActHistoryAddress"
       responses:
         "200":
-          description: Chunk exists
+          description: Headers for the content at the reference.
           headers:
             Content-Type:
               description: The MIME type of the resource.

pkg/api/bytes_test.go

@@ -507,9 +507,9 @@ func TestBytesHeadErrorsMatchGet(t *testing.T) {
 		ref  string
 		want int
 	}{
-		{"unresolvable address", "abcd", http.StatusInternalServerError},
+		{"short address", "abcd", http.StatusInternalServerError},
 		{"unknown address", "0000000000000000000000000000000000000000000000000000000000000001", http.StatusNotFound},
-		{"invalid address", "zzzz", http.StatusBadRequest},
+		{"non-hex address", "zzzz", http.StatusBadRequest},
 	}
 
 	for _, tt := range tests {

pkg/api/bzz.go

@@ -794,7 +794,7 @@ func (s *Service) downloadHandler(logger log.Logger, w http.ResponseWriter, r *h
 	w.Header().Add(AccessControlExposeHeaders, ContentDispositionHeader)
 
 	if headersOnly {
-		// http.ServeContent sets this on the GET path, but is not reached here.
+		// http.ServeContent would set this, but the GET path is not reached here.
 		// "bytes" is the range unit, not the endpoint.
 		w.Header().Set(AcceptRangesHeader, "bytes")
 		w.Header().Add(AccessControlExposeHeaders, AcceptRangesHeader)

pkg/api/bzz_test.go

@@ -597,6 +597,7 @@ func TestBzzFiles(t *testing.T) {
 				jsonhttptest.WithRequestBody(bytes.NewReader(simpleData)),
 				jsonhttptest.WithRequestHeader(api.ContentTypeHeader, "text/html; charset=utf-8"),
 				jsonhttptest.WithExpectedContentLength(21),
+				jsonhttptest.WithExpectedResponseHeader(api.AcceptRangesHeader, "bytes"),
 			)
 		})
 	})