Merge pull request #15877 from openshift-cherrypick-robot/cherry-pick-15822-to-release-4.19

[release-4.19] OCPBUGS-70135: Update client initialization in checkPackageManifestHandler

Author: openshift-merge-bot[bot]

pkg/olm/handler.go

@@ -82,7 +82,7 @@ func (o *OLMHandler) OperandsList(user *auth.User, w http.ResponseWriter, r *htt
 	w.Write(resp)
 }
 
-func (o *OLMHandler) CheckPackageManifest(w http.ResponseWriter, r *http.Request) {
+func (o *OLMHandler) CheckPackageManifest(user *auth.User, w http.ResponseWriter, r *http.Request) {
 	if r.Method != "GET" {
 		w.Header().Set("Allow", "GET")
 		serverutils.SendResponse(w, http.StatusMethodNotAllowed, serverutils.ApiError{Err: "Method unsupported, the only supported methods is GET"})
@@ -96,7 +96,7 @@ func (o *OLMHandler) CheckPackageManifest(w http.ResponseWriter, r *http.Request
 		return
 	}
 
-	client, _, err := o.getClientWithScheme(nil)
+	client, _, err := o.getClientWithScheme(user)
 	if err != nil {
 		klog.Error(err)
 		serverutils.SendResponse(w, http.StatusInternalServerError, serverutils.ApiError{Err: fmt.Sprint(err)})

pkg/server/server.go

@@ -469,7 +469,9 @@ func (s *Server) HTTPHandler() (http.Handler, error) {
 
 	handle(packageManifestEndpoint, http.StripPrefix(
 		proxy.SingleJoiningSlash(s.BaseURL.Path, packageManifestEndpoint),
-		authHandler(olmHandler.CheckPackageManifest),
+		authHandlerWithUser(func(user *auth.User, w http.ResponseWriter, r *http.Request) {
+			olmHandler.CheckPackageManifest(user, w, r)
+		}),
 	))
 
 	handle(operandsListEndpoint, http.StripPrefix(