header: move from internal to pkg

pkg/authn/authn.go

@@ -3,7 +3,7 @@ package authn
 import (
 	"net/http"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 )
 
 // Authenticator middleware

pkg/authn/basic.go

@@ -6,7 +6,7 @@ import (
 	"net/http"
 	"net/url"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 )
 
 var (

pkg/authn/forward.go

@@ -7,7 +7,7 @@ import (
 	"net/http"
 	"net/url"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 	"github.com/moonrhythm/parapet/pkg/internal/pool"
 )
 

pkg/compress/compress.go

@@ -10,7 +10,7 @@ import (
 	"strings"
 	"sync"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 )
 
 // Compress is the compress middleware

pkg/cors/cors.go

@@ -6,7 +6,7 @@ import (
 	"strings"
 	"time"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 )
 
 // New creates new default cors middleware for public api

pkg/internal/header/header.go → pkg/header/const.go

@@ -1,9 +1,5 @@
 package header
 
-import (
-	"net/http"
-)
-
 // Headers in canonical format
 const (
 	AcceptEncoding                = "Accept-Encoding"
@@ -34,55 +30,3 @@ const (
 	XRealIP                       = "X-Real-Ip"
 	XRequestID                    = "X-Request-Id"
 )
-
-func AddIfNotExists(h http.Header, key, value string) {
-	for _, v := range h[key] {
-		if v == value {
-			return
-		}
-	}
-	h[key] = append(h[key], value)
-}
-
-func Get(h http.Header, key string) string {
-	if h == nil {
-		return ""
-	}
-	v := h[key]
-	if len(v) == 0 {
-		return ""
-	}
-	return v[0]
-}
-
-func Exists(h http.Header, key string) bool {
-	if h == nil {
-		return false
-	}
-	v := h[key]
-	if len(v) == 0 {
-		return false
-	}
-	return v[0] != ""
-}
-
-func Del(h http.Header, key string) {
-	if h == nil {
-		return
-	}
-	delete(h, key)
-}
-
-func Set(h http.Header, key, value string) {
-	if h == nil {
-		return
-	}
-	h[key] = []string{value}
-}
-
-func Add(h http.Header, key, value string) {
-	if h == nil {
-		return
-	}
-	h[key] = append(h[key], value)
-}

pkg/internal/header/header_test.go → pkg/header/const_test.go

@@ -6,7 +6,7 @@ import (
 
 	"github.com/stretchr/testify/assert"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 )
 
 func TestHeaders(t *testing.T) {

pkg/header/header.go

@@ -0,0 +1,57 @@
+package header
+
+import (
+	"net/http"
+)
+
+func AddIfNotExists(h http.Header, key, value string) {
+	for _, v := range h[key] {
+		if v == value {
+			return
+		}
+	}
+	h[key] = append(h[key], value)
+}
+
+func Get(h http.Header, key string) string {
+	if h == nil {
+		return ""
+	}
+	v := h[key]
+	if len(v) == 0 {
+		return ""
+	}
+	return v[0]
+}
+
+func Exists(h http.Header, key string) bool {
+	if h == nil {
+		return false
+	}
+	v := h[key]
+	if len(v) == 0 {
+		return false
+	}
+	return v[0] != ""
+}
+
+func Del(h http.Header, key string) {
+	if h == nil {
+		return
+	}
+	delete(h, key)
+}
+
+func Set(h http.Header, key, value string) {
+	if h == nil {
+		return
+	}
+	h[key] = []string{value}
+}
+
+func Add(h http.Header, key, value string) {
+	if h == nil {
+		return
+	}
+	h[key] = append(h[key], value)
+}

pkg/hsts/hsts.go

@@ -5,7 +5,7 @@ import (
 	"strconv"
 	"time"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 )
 
 // HSTS middleware

pkg/logger/logger.go

@@ -10,7 +10,7 @@ import (
 	"os"
 	"time"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 )
 
 // Logger middleware

pkg/ratelimit/ratelimit.go

@@ -7,7 +7,7 @@ import (
 	"strconv"
 	"time"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 )
 
 // New creates new rate limiter

pkg/redirect/https.go

@@ -3,7 +3,7 @@ package redirect
 import (
 	"net/http"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 )
 
 // HTTPS creates new https redirector

pkg/redirect/nonwww.go

@@ -4,7 +4,7 @@ import (
 	"net/http"
 	"strings"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 )
 
 // NonWWW creates new non www redirector

pkg/redirect/www.go

@@ -4,7 +4,7 @@ import (
 	"net/http"
 	"strings"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 )
 
 // WWW creates new www redirector

pkg/requestid/requestid.go

@@ -5,7 +5,7 @@ import (
 
 	"github.com/gofrs/uuid"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 	"github.com/moonrhythm/parapet/pkg/logger"
 )
 

pkg/stackdriver/trace.go

@@ -12,7 +12,7 @@ import (
 	"go.opencensus.io/trace/propagation"
 	"google.golang.org/api/option"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 )
 
 // NewTrace creates new stack driver trace middleware

pkg/upstream/transport.go

@@ -11,7 +11,7 @@ import (
 
 	"golang.org/x/net/http2"
 
-	"github.com/moonrhythm/parapet/pkg/internal/header"
+	"github.com/moonrhythm/parapet/pkg/header"
 )
 
 const (