fix: allowing domain to be added including a scheme (#72)

Author: Skarlso

apis/mpas/v1alpha1/repository_types.go

@@ -48,7 +48,10 @@ type RepositorySpec struct {
 	Visibility string `json:"visibility,omitempty"`
 	//+kubebuilder:default:=true
 	IsOrganization bool `json:"isOrganization"`
+	// Domain specifies an optional domain address to be used instead of the defaults like github.com.
+	// Must NOT contain the scheme.
 	//+optional
+	//+kubebuilder:validation:Pattern="^\\w+(\\.|:[0-9]).*$"
 	Domain string `json:"domain,omitempty"`
 	//+optional
 	Maintainers []string `json:"maintainers,omitempty"`
@@ -102,7 +105,7 @@ func (in Repository) GetRepositoryURL() string {
 			return fmt.Sprintf("%s:%s/%s", in.Spec.Domain, in.Spec.Owner, in.GetName())
 		}
 
-		return fmt.Sprintf("%s/%s/%s", in.Spec.Domain, in.Spec.Owner, in.GetName())
+		return fmt.Sprintf("https://%s/%s/%s", in.Spec.Domain, in.Spec.Owner, in.GetName())
 	}
 
 	domain := ""

config/crd/bases/mpas.ocm.software_repositories.yaml

@@ -71,6 +71,9 @@ spec:
                 default: main
                 type: string
               domain:
+                description: Domain specifies an optional domain address to be used
+                  instead of the defaults like github.com. Must NOT contain the scheme.
+                pattern: ^\w+(\.|:[0-9]).*$
                 type: string
               existingRepositoryPolicy:
                 default: adopt

pkg/providers/github/github.go

@@ -88,9 +88,9 @@ func (c *Client) CreateBranchProtection(ctx context.Context, obj mpasv1alpha1.Re
 	if err != nil {
 		return fmt.Errorf("failed to retrieve token: %w", err)
 	}
+
 	ts := oauth2.StaticTokenSource(&oauth2.Token{AccessToken: string(token)})
 	tc := oauth2.NewClient(context.Background(), ts)
-
 	g := ggithub.NewClient(tc)
 
 	if _, _, err := g.Repositories.UpdateBranchProtection(ctx, obj.Spec.Owner, obj.Name, obj.Spec.DefaultBranch, &ggithub.ProtectionRequest{