Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: update README and logging with information on new mask-password input #492

Merged
merged 4 commits into from
Aug 9, 2023

Conversation

arjraman
Copy link
Contributor

@arjraman arjraman commented Aug 8, 2023

Issue #, if available:
#491

Description of changes:
Update README and logging with information on new mask-password input.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@mergify mergify bot merged commit 101a971 into main Aug 9, 2023
6 checks passed
@ecr-cicd-bot ecr-cicd-bot deleted the arjraman branch August 9, 2023 19:12
@pascalgulikers
Copy link

Did you ever read this? #483 (comment)

@pascalgulikers
Copy link

In our case we need the Docker credentials in order to pull an image from ecr in the following job, by showing this warning the credentials are more in the picture then before or as a Dutch saying goes: "Waking up sleeping dogs"

@pascalgulikers
Copy link

@arjraman is it possible to remove the warnings? Because if you need unmasked values to pass to other jobs, you don't want to show that the credentials are unmasked?

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

Successfully merging this pull request may close these issues.

4 participants