repair GetFileContent: returns a raw byte response (ktrysmt#226)

alex-mj · alex-mazhaika · web-flow · commit bcef472e9f41 · 2022-12-07T16:49:37.000+09:00
Co-authored-by: alexander_mazhaika &lt;alexander.mazhaika@endotech.io&gt;
diff --git a/repository.go b/repository.go
@@ -309,17 +309,13 @@ func (r *Repository) GetFileContent(ro *RepositoryFilesOptions) ([]byte, error)
 		return nil, err
 	}
 
-	response, err := r.c.execute("GET", urlStr, "")
+	resBody, err := r.c.executeRaw("GET", urlStr, "")
 	if err != nil {
 		return nil, err
 	}
+	defer resBody.Close()
 
-	content, ok := response.([]byte)
-	if !ok {
-		return nil, fmt.Errorf("requested path is not a file")
-	}
-
-	return content, nil
+	return ioutil.ReadAll(resBody)
 }
 
 func (r *Repository) ListFiles(ro *RepositoryFilesOptions) ([]RepositoryFile, error) {

Original file line number	Diff line number	Diff line change
`@@ -309,17 +309,13 @@ func (r Repository) GetFileContent(ro RepositoryFilesOptions) ([]byte, error)`
`309`	`309`	`return nil, err`
`310`	`310`	`}`
`311`	`311`
`312`		`- response, err := r.c.execute("GET", urlStr, "")`
	`312`	`+ resBody, err := r.c.executeRaw("GET", urlStr, "")`
`313`	`313`	`if err != nil {`
`314`	`314`	`return nil, err`
`315`	`315`	`}`
	`316`	`+ defer resBody.Close()`
`316`	`317`
`317`		`- content, ok := response.([]byte)`
`318`		`- if !ok {`
`319`		`- return nil, fmt.Errorf("requested path is not a file")`
`320`		`- }`
`321`		`-`
`322`		`- return content, nil`
	`318`	`+ return ioutil.ReadAll(resBody)`
`323`	`319`	`}`
`324`	`320`
`325`	`321`	`func (r Repository) ListFiles(ro RepositoryFilesOptions) ([]RepositoryFile, error) {`