Method azureIterator.Next has a Cognitive Complexity of 57 (exceeds 20 allowed). Consider refactoring.

Open

func (a *azureIterator) Next() (key vault.StoredKey, err error) {
    if a.done {
        return nil, vault.ErrDone
    }

• Read up
  Read up

Method azureIterator.Next has 72 lines of code (exceeds 50 allowed). Consider refactoring.

Open

func (a *azureIterator) Next() (key vault.StoredKey, err error) {
    if a.done {
        return nil, vault.ErrDone
    }

Method azureIterator.Next has 11 return statements (exceeds 4 allowed).

Open

func (a *azureIterator) Next() (key vault.StoredKey, err error) {
    if a.done {
        return nil, vault.ErrDone
    }

Method Vault.Import has 52 lines of code (exceeds 50 allowed). Consider refactoring.

Open

func (v *Vault) Import(ctx context.Context, priv crypt.PrivateKey, opt utils.Options) (vault.StoredKey, error) {
    keyName, ok, err := opt.GetString("name")
    if err != nil {
        return nil, fmt.Errorf("(Azure/%s): %w", v.config.Vault, err)
    }

Method Vault.Import has 9 return statements (exceeds 4 allowed).

Open

func (v *Vault) Import(ctx context.Context, priv crypt.PrivateKey, opt utils.Options) (vault.StoredKey, error) {
    keyName, ok, err := opt.GetString("name")
    if err != nil {
        return nil, fmt.Errorf("(Azure/%s): %w", v.config.Vault, err)
    }

Method Vault.SignMessage has 8 return statements (exceeds 4 allowed).

Open

func (v *Vault) SignMessage(ctx context.Context, message []byte, key vault.StoredKey) (crypt.Signature, error) {
    digest := crypt.DigestFunc(message)
    azureKey, ok := key.(*azureKey)
    if !ok {
        return nil, errors.Wrap(fmt.Errorf("(Azure/%s): not a Azure key: %T", v.config.Vault, key), http.StatusBadRequest)

Avoid deeply nested control flow statements.

Open

                    if err != nil {
                        return nil, fmt.Errorf("(Azure/%s): %w", a.v.config.Vault, err)
                    }

Avoid deeply nested control flow statements.

Open

                    if status != 0 {
                        err = errors.Wrap(err, status)
                    }

Method Vault.request has 7 return statements (exceeds 4 allowed).

Open

func (v *Vault) request(ctx context.Context, client *http.Client, method, url string, body io.Reader, result interface{}) (status int, err error) {
    req, err := http.NewRequestWithContext(ctx, method, url, body)
    if err != nil {
        return status, err
    }

Avoid deeply nested control flow statements.

Open

                    if u == "" {
                        a.done = true
                        return nil, vault.ErrDone
                    }

Method Vault.request has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

func (v *Vault) request(ctx context.Context, client *http.Client, method, url string, body io.Reader, result interface{}) (status int, err error) {

Method Vault.GetPublicKey has 5 return statements (exceeds 4 allowed).

Open

func (v *Vault) GetPublicKey(ctx context.Context, keyID string) (vault.StoredKey, error) {
    u, err := v.makeURL(keyID, "")
    if err != nil {
        return nil, fmt.Errorf("(Azure/%s): %w", v.config.Vault, err)
    }

Method Vault.Ready has 5 return statements (exceeds 4 allowed).

Open

func (v *Vault) Ready(ctx context.Context) (bool, error) {
    if v.managementClient == nil {
        return true, nil // ignore
    }

Similar blocks of code found in 5 locations. Consider refactoring.

Open

func init() {
    vault.RegisterVault("azure", func(ctx context.Context, node *yaml.Node) (vault.Vault, error) {
        var conf Config
        if node == nil || node.Kind == 0 {
            return nil, errors.New("(Azure): config is missing")

• Read up
  Read up