Use RawJSON type for Ext instead of interface{}

As proposed in: #8

README.md

@@ -1,6 +1,6 @@
 # openrtb
 
-[![GoDoc](https://godoc.org/gopkg.in/mxmCherry/openrtb.v2?status.svg)](https://godoc.org/gopkg.in/mxmCherry/openrtb.v2)
+[![GoDoc](https://godoc.org/gopkg.in/mxmCherry/openrtb.v3?status.svg)](https://godoc.org/gopkg.in/mxmCherry/openrtb.v3)
 
 [OpenRTB](//github.com/openrtb/OpenRTB) [v2.3.1](//github.com/openrtb/OpenRTB/blob/master/OpenRTB-API-Specification-Version-2-3-1-FINAL.pdf) types for Go programming language (golang)
 
@@ -9,11 +9,11 @@
 # Using
 
 ```bash
-go get -u "gopkg.in/mxmCherry/openrtb.v2"
+go get -u "gopkg.in/mxmCherry/openrtb.v3"
 ```
 
 ```go
-import "gopkg.in/mxmCherry/openrtb.v2"
+import "gopkg.in/mxmCherry/openrtb.v3"
 ```
 
 # Goals

app.go

@@ -128,5 +128,5 @@ type App struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

banner.go

@@ -151,5 +151,5 @@ type Banner struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

bid.go

@@ -152,5 +152,5 @@ type Bid struct {
 	//   object
 	// Description:
 	//   Placeholder for bidder-specific extensions to OpenRTB
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

bid_request.go

@@ -150,5 +150,5 @@ type BidRequest struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

bid_response.go

@@ -70,5 +70,5 @@ type BidResponse struct {
 	//   object
 	// Description:
 	//   Placeholder for bidder-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

content.go

@@ -176,5 +176,5 @@ type Content struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

data.go

@@ -39,5 +39,5 @@ type Data struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

deal.go

@@ -66,5 +66,5 @@ type Deal struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

device.go

@@ -242,5 +242,5 @@ type Device struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

geo.go

@@ -100,5 +100,5 @@ type Geo struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

imp.go

@@ -128,5 +128,5 @@ type Imp struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

native.go

@@ -57,5 +57,5 @@ type Native struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

pmp.go

@@ -30,5 +30,5 @@ type PMP struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

producer.go

@@ -48,5 +48,5 @@ type Producer struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

publisher.go

@@ -45,5 +45,5 @@ type Publisher struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

raw_json.go

@@ -0,0 +1,25 @@
+package openrtb
+
+import "errors"
+
+// RawJSON is a raw encoded JSON value.
+// It implements encoding/json.Marshaler and encoding/json.Unmarshaler and can
+// be used to delay JSON decoding or precompute a JSON encoding.
+//
+// Basically, it's just a copy of encoding/json.RawMessage type,
+// but with more convenient non-pointer encoding.
+type RawJSON []byte
+
+// MarshalJSON returns m as the JSON encoding of m.
+func (m RawJSON) MarshalJSON() ([]byte, error) {
+	return m, nil
+}
+
+// UnmarshalJSON sets *m to a copy of data.
+func (m *RawJSON) UnmarshalJSON(data []byte) error {
+	if m == nil {
+		return errors.New("openrtb.RawJSON: UnmarshalJSON on nil pointer")
+	}
+	*m = append((*m)[0:0], data...)
+	return nil
+}

raw_json_test.go

@@ -0,0 +1,57 @@
+package openrtb_test
+
+import (
+	"encoding/json"
+
+	"github.com/mxmCherry/openrtb"
+
+	. "github.com/onsi/ginkgo"
+	. "github.com/onsi/gomega"
+)
+
+var _ = Describe("RawJSON", func() {
+
+	var _ json.Marshaler = (openrtb.RawJSON)(nil)
+	var _ json.Unmarshaler = (*openrtb.RawJSON)(nil)
+
+	It("should encode JSON", func() {
+		subject := openrtb.RawJSON(`true`)
+
+		actual, err := subject.MarshalJSON()
+		Expect(err).NotTo(HaveOccurred())
+		Expect(actual).To(Equal([]byte(`true`)))
+	})
+
+	It("should decode JSON", func() {
+		subject := openrtb.RawJSON(nil)
+
+		err := subject.UnmarshalJSON([]byte(`true`))
+		Expect(err).NotTo(HaveOccurred())
+		Expect(subject).To(Equal(openrtb.RawJSON(`true`)))
+	})
+
+	It("should decode JSON when embedded into struct", func() {
+		wrapper := struct {
+			Raw openrtb.RawJSON `json:"raw"`
+		}{
+			Raw: nil,
+		}
+
+		err := json.Unmarshal([]byte(`{"raw":true}`), &wrapper)
+		Expect(err).NotTo(HaveOccurred())
+		Expect(wrapper.Raw).To(Equal(openrtb.RawJSON(`true`)))
+	})
+
+	It("should encode JSON when embedded into struct", func() {
+		wrapper := struct {
+			Raw openrtb.RawJSON `json:"raw"`
+		}{
+			Raw: openrtb.RawJSON(`true`),
+		}
+
+		actual, err := json.Marshal(wrapper)
+		Expect(err).NotTo(HaveOccurred())
+		Expect(actual).To(MatchJSON(`{"raw":true}`))
+	})
+
+})

regs.go

@@ -22,5 +22,5 @@ type Regs struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

seat_bid.go

@@ -40,5 +40,5 @@ type SeatBid struct {
 	//   object
 	// Description:
 	//   Placeholder for bidder-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

segment.go

@@ -37,5 +37,5 @@ type Segment struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

site.go

@@ -127,5 +127,5 @@ type Site struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

user.go

@@ -100,5 +100,5 @@ type User struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }

video.go

@@ -212,5 +212,5 @@ type Video struct {
 	//   object
 	// Description:
 	//   Placeholder for exchange-specific extensions to OpenRTB.
-	Ext Ext `json:"ext,omitempty"`
+	Ext RawJSON `json:"ext,omitempty"`
 }