Skip to content
This repository has been archived by the owner on Jun 23, 2020. It is now read-only.

Empty secret places ******** between every character #67

Closed
johncrenshaw opened this issue Apr 19, 2019 · 1 comment
Closed

Empty secret places ******** between every character #67

johncrenshaw opened this issue Apr 19, 2019 · 1 comment

Comments

@johncrenshaw
Copy link

Using any simple drone.yml, such as:

kind: pipeline
name: default

steps:
- name: Hello World
  image: "alpine:3.8"
  commands:
  - echo "Hello World"

And a secret.txt file with an empty secret such as:

FOO=""

The output for the following command is virtually unreadable:

drone exec --secret-file="secret.txt" --trusted

The problem is the replace rules prepared for writing lines, which adds a rule to replace nothing with ********, resulting in ******** being inserted after every character of output. An empty string check in the newReplacer function at the end of https://github.com/drone/drone-runtime/blob/master/runtime/line.go would presumably fix the problem.

@bradrydzewski bradrydzewski transferred this issue from harness/drone-cli Apr 19, 2019
@bradrydzewski
Copy link
Member

this was fixed by the runner-go library, which is used by all new drone/drone-runner-* images

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants