Merge pull request #19 from KusionStack/fix-source-bug

fix: some bug for source
KusionStack · May 26, 2022 · a184410 · a184410
2 parents 489bde2 + 6eae537
commit a184410
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/pkg/sources/cdn/source.go b/pkg/sources/cdn/source.go
@@ -21,7 +21,7 @@ func (s *releaseSource) GetVersions() []string {
 	versions := []string{"latest"}
 
 	// Get tags from github repo of kusion
-	remoteURL := "git@github.com:KusionStack/kusion"
+	remoteURL := "https://github.com/KusionStack/kusion"
 
 	tags, err := gitutil.GetTagListFromRemote(remoteURL, true)
 	if err != nil {
@@ -41,8 +41,9 @@ func (s *releaseSource) GetVersions() []string {
 }
 
 func (s *releaseSource) GetDownloadURL(ver string) (string, error) {
-	if ver == "latest" {
-		return getArchiveDownloadURL(s.GetVersions()[1])
+	vers := s.GetVersions()
+	if ver == "latest" && len(vers) > 0 {
+		return getArchiveDownloadURL(vers[1])
 	}
 
 	return getArchiveDownloadURL(ver)

diff --git a/pkg/sources/github/source.go b/pkg/sources/github/source.go
@@ -21,7 +21,7 @@ func (s *releaseSource) GetVersions() []string {
 	versions := []string{"latest"}
 
 	// Get tags from github repo of kusion
-	remoteURL := "git@github.com:KusionStack/kusion"
+	remoteURL := "https://github.com/KusionStack/kusion"
 
 	tags, err := gitutil.GetTagListFromRemote(remoteURL, true)
 	if err != nil {
@@ -41,8 +41,9 @@ func (s *releaseSource) GetVersions() []string {
 }
 
 func (s *releaseSource) GetDownloadURL(ver string) (string, error) {
-	if ver == "latest" {
-		return getArchiveDownloadURL(s.GetVersions()[1])
+	vers := s.GetVersions()
+	if ver == "latest" && len(vers) > 0 {
+		return getArchiveDownloadURL(vers[1])
 	}
 
 	return getArchiveDownloadURL(ver)