elastic / elastic/ecctl

Store api key in keychain or credential manager instead of plaintext

Open
#766 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
58
Forks
61
PR merge metrics
No merged PRs in 30d

Description

## Overview

Per a discussion with @rseldner we noticed ecctl stores API key in plaintext.
This is insecure.

## Possible Implementation

We would propose to store key in keychain (macOS) or credential manager (WinOS), etc.
If hard, then avoid saving it in `$HOME/.ecctl/config.json`, but let user specify it in env variables or bash_profiles, to reduce risk.

## Testing

n/a

## Context

given api key is extremely important to ops the deployment, we should keep it safe and secure

## Your Environment

macOS

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.