fix: added some validations and docs

docs/configuration/environment_variables.md

@@ -12,7 +12,7 @@ The helm chart populate them using [values.yaml](https://github.com/quickube/pip
   The git token that will be used.
 
 - GIT_URL
-  the git url that will be used, mostly relevant when running gitlab on-prem
+  the git url that will be used, only relevant when running gitlab self hosted
 
 - GIT_ORG_NAME
   The organization name.

pkg/conf/git_provider.go

@@ -9,7 +9,7 @@ import (
 type GitProviderConfig struct {
 	Provider           string `envconfig:"GIT_PROVIDER" required:"true"`
 	Token              string `envconfig:"GIT_TOKEN" required:"true"`
-	Url			       string `envconfig:"GIT_URL" required:"true"`
+	Url			       string `envconfig:"GIT_URL" required:"false"`
 	OrgName           string `envconfig:"GIT_ORG_NAME" required:"true"`
 	OrgLevelWebhook    bool   `envconfig:"GIT_ORG_LEVEL_WEBHOOK" default:"false" required:"false"`
 	RepoList           string `envconfig:"GIT_WEBHOOK_REPO_LIST" required:"false"`

pkg/git_provider/gitlab.go

@@ -41,6 +41,8 @@ func NewGitlabClient(cfg *conf.GlobalConfig) (Client, error) {
 		return nil, fmt.Errorf("failed to get organization data %s", resp.Status)
 	}
 
+	cfg = EnsureGitlabURL(cfg)
+
 	c := &GitlabClientImpl{
 		client: client,
 		cfg:    cfg,

pkg/git_provider/gitlab_utils.go

@@ -153,4 +153,12 @@ func FixRepoNames(c *GitlabClientImpl) error{
 	}
 	c.cfg.GitProviderConfig.RepoList = strings.Join(formattedRepos, ",")
 	return nil
+}
+
+func EnsureGitlabURL(c *conf.GlobalConfig) *conf.GlobalConfig{
+	saasGitlabUrl := "https://gitlab.com"
+	if c.Url == "" {
+		c.Url = saasGitlabUrl
+	}
+	return c
 }