dkirillov/distribution
1
0
Fork 0
forked from TrueCloudLab/distribution
Code Pull requests Activity

Fixes #2835 Process Accept header MIME types in case-insensitive way

Browse source 
Use mime.ParseMediaType to parse the media types in Accept header in manifest request. Ignore the failed ones.

Signed-off-by: Yu Wang <yuwa@microsoft.com>
This commit is contained in:
Yu Wang 2019-02-21 14:07:57 -08:00
parent 0d3efadf01
commit a683c7c235
1 changed files with 3 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
registry/handlers/manifests.go
View file

@ -3,6 +3,7 @@ package handlers
import ( import (
"bytes" "bytes"
"fmt" "fmt"
"mime"
"net/http" "net/http"
"strings" "strings"
@ -97,14 +98,10 @@ func (imh *manifestHandler) GetManifest(w http.ResponseWriter, r *http.Request)
// we need to split each header value on "," to get the full list of "Accept" values (per RFC 2616) // we need to split each header value on "," to get the full list of "Accept" values (per RFC 2616)
// https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.1 // https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.1
for _, mediaType := range strings.Split(acceptHeader, ",") { for _, mediaType := range strings.Split(acceptHeader, ",") {
// remove "; q=..." if present if mediaType, _, err = mime.ParseMediaType(mediaType); err != nil {
if i := strings.Index(mediaType, ";"); i >= 0 { continue
mediaType = mediaType[:i]
} }
// it's common (but not required) for Accept values to be space separated ("a/b, c/d, e/f")
mediaType = strings.TrimSpace(mediaType)
if mediaType == schema2.MediaTypeManifest { if mediaType == schema2.MediaTypeManifest {
supports[manifestSchema2] = true supports[manifestSchema2] = true
} }