Remove need for auth header for versions endpoint

[#165573409]

Signed-off-by: Ka Hin Ng <[email protected]>

pivnet.go

@@ -180,17 +180,20 @@ func (c Client) CreateRequest(
  return nil, err
  }
 
- accessToken, err := c.token.AccessToken()
- if err != nil {
- return nil, err
- }
+ if !isVersionsEndpoint(endpoint) {
+ accessToken, err := c.token.AccessToken()
+ if err != nil {
+ return nil, err
+ }
 
- authorizationHeader, err := AuthorizationHeader(accessToken)
- if err != nil {
- return nil, fmt.Errorf("could not create authorization header: %s", err)
+ authorizationHeader, err := AuthorizationHeader(accessToken)
+ if err != nil {
+ return nil, fmt.Errorf("could not create authorization header: %s", err)
+ }
+
+ req.Header.Add("Authorization", authorizationHeader)
  }
 
- req.Header.Add("Authorization", authorizationHeader)
  req.Header.Add("Content-Type", "application/json")
  req.Header.Add("User-Agent", c.userAgent)
 
@@ -283,3 +286,7 @@ func (c Client) handleUnexpectedResponse(resp *http.Response) error {
  }
  }
 }
+
+func isVersionsEndpoint(endpoint string) bool {
+ return endpoint == "/versions"
+}

pivnet_test.go

@@ -503,5 +503,17 @@ var _ = Describe("PivnetClient", func() {
  Expect(err).NotTo(HaveOccurred())
  Expect(req.URL.Path).To(Equal("/api/v2/foo/bar"))
  })
+
+ It("does not add auth header if versions endpoint", func() {
+ req, err := client.CreateRequest(
+ "GET",
+ "/versions",
+ nil,
+ )
+
+ Expect(err).NotTo(HaveOccurred())
+ Expect(req.Header.Get("Authorization")).To(Equal(""))
+ Expect(req.Header.Get("Content-Type")).To(Equal("application/json"))
+ })
  })
 })