Merge pull request #278 from fredbi/fix/277-header-ext

Fix/277 header ext

Author: fredbi

.golangci.yml

@@ -4,7 +4,10 @@ linters:
   disable:
     - depguard
     - funlen
+    - goconst
     - godox
+    - gomodguard
+    - gomodguard_v2
     - exhaustruct
     - nlreturn
     - nonamedreturns

debug_test.go

@@ -38,7 +38,7 @@ func TestDebug(t *testing.T) {
 	Debug = false
 	_ = tmpFile.Close()
 
-	flushed, _ := os.Open(tmpName) //nolint:gosec // test file, path is from os.CreateTemp
+	flushed, _ := os.Open(tmpName)
 	buf := make([]byte, 500)
 	_, _ = flushed.Read(buf)
 	specLogger.SetOutput(os.Stdout)

header.go

@@ -150,7 +150,11 @@ func (h Header) MarshalJSON() ([]byte, error) {
 	if err != nil {
 		return nil, err
 	}
-	return jsonutils.ConcatJSON(b1, b2, b3), nil
+	b4, err := json.Marshal(h.VendorExtensible)
+	if err != nil {
+		return nil, err
+	}
+	return jsonutils.ConcatJSON(b1, b2, b3, b4), nil
 }
 
 // UnmarshalJSON unmarshals this header from JSON.

header_test.go

@@ -75,6 +75,7 @@ const headerJSON = `{
 
 func TestIntegrationHeader(t *testing.T) {
 	assert.JSONUnmarshalAsT(t, header, headerJSON)
+	assert.JSONMarshalAsT(t, headerJSON, header)
 }
 
 func TestJSONLookupHeader(t *testing.T) {

schema_loader.go

@@ -117,7 +117,7 @@ func (r *schemaLoader) updateBasePath(transitive *schemaLoader, basePath string)
 
 func (r *schemaLoader) resolveRef(ref *Ref, target any, basePath string) error {
 	tgt := reflect.ValueOf(target)
-	if tgt.Kind() != reflect.Ptr {
+	if tgt.Kind() != reflect.Pointer {
 		return ErrResolveRefNeedsAPointer
 	}