hasura/graphql-engine
1
0
Fork 0
mirror of https://github.com/hasura/graphql-engine.git synced 2024-10-04 22:07:40 +03:00
Code Issues Projects Releases Wiki Activity

cli: refactor internal/cliext to use internal/errors

Browse Source 
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6695
GitOrigin-RevId: 11c1519e082103775ce3ffb1e2a95c766a299469
This commit is contained in:
Mohd Bilal 2022-11-10 21:10:39 +05:30 committed by hasura-bot
parent 4c24e6d56b
commit 8e2a18b256
1 changed files with 19 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
cli/internal/cliext/cli_ext.go
View File

@ -10,13 +10,18 @@ import (
"runtime"
"github.com/hasura/graphql-engine/cli/v2"
"github.com/pkg/errors"
"github.com/hasura/graphql-engine/cli/v2/internal/errors"
)
func getCliExtFileContent(ec *cli.ExecutionContext) ([]byte, error) {
var op errors.Op = "cliext.getCliExtFileContent"
if ec.CliExtSourceBinPath != "" {
ec.Logger.Debug("cli-ext: setting up using --cli-ext-path binary")
return ioutil.ReadFile(ec.CliExtSourceBinPath)
b, err := ioutil.ReadFile(ec.CliExtSourceBinPath)
if err != nil {
return b, errors.E(op, err)
}
return b, nil
}
var cliExtBinName string = "cli-ext"
@ -39,25 +44,30 @@ func getCliExtFileContent(ec *cli.ExecutionContext) ([]byte, error) {
)
resp, err := http.Get(CDNpath)
if err != nil {
return nil, err
return nil, errors.E(op, err)
}
defer resp.Body.Close()
if resp.StatusCode != http.StatusOK {
return nil, fmt.Errorf("error downloading cli-ext from CDN %s with status code: %d", CDNpath, resp.StatusCode)
return nil, errors.E(op, fmt.Errorf("error downloading cli-ext from CDN %s with status code: %d", CDNpath, resp.StatusCode))
}
return ioutil.ReadAll(bufio.NewReader(resp.Body))
b, err := ioutil.ReadAll(bufio.NewReader(resp.Body))
if err != nil {
return b, errors.E(op, err)
}
return b, nil
}
// Setup sets up cli-ext binary for using it in various cli commands
func Setup(ec *cli.ExecutionContext) error {
var op errors.Op = "cliext.Setup"
parentDirPath := filepath.Join(ec.GlobalConfigDir, "cli-ext", ec.Version.GetCLIVersion())
err := os.MkdirAll(parentDirPath, 0755)
if err != nil {
return errors.Wrapf(err, "error creating base directory while setting up cli-ext")
return errors.E(op, fmt.Errorf("error creating base directory while setting up cli-ext: %w", err))
}
cliExtDirPath, err := ioutil.TempDir(parentDirPath, "cli-ext-*")
if err != nil {
return errors.Wrapf(err, "error creating directory while setting up cli-ext")
return errors.E(op, fmt.Errorf("error creating directory while setting up cli-ext: %w", err))
}
ec.CliExtDestinationDir = cliExtDirPath
@ -68,11 +78,11 @@ func Setup(ec *cli.ExecutionContext) error {
cliExtBinPath := filepath.Join(cliExtDirPath, cliExtBinName)
cliExtFileContent, err := getCliExtFileContent(ec)
if err != nil {
return fmt.Errorf("error reading cli-ext binary: %w", err)
return errors.E(op, fmt.Errorf("error reading cli-ext binary: %w", err))
}
err = ioutil.WriteFile(cliExtBinPath, cliExtFileContent, 0755)
if err != nil {
return errors.Wrap(err, "error unpacking binary while setting up cli-ext")
return errors.E(op, fmt.Errorf("error unpacking binary while setting up cli-ext: %w", err))
}
ec.CliExtDestinationBinPath = cliExtBinPath