Skip to content

adding warning if users are going to print to the command line #40

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.

Sign up for GitHub

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

Merged
SethHollandsworth merged 1 commit into from
Nov 1, 2023
Merged

adding warning if users are going to print to the command line #40

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions src/confcom/azext_confcom/custom.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,16 @@ def acipolicygen_confcom(
elif save_to_file and arm_template and not (print_policy_to_terminal or outraw or outraw_pretty_print):
error_out("Must print policy to terminal when saving to file")

if print_existing_policy or outraw or outraw_pretty_print:
logger.warning(
"%s %s %s %s %s",
"Secrets that are included in the provided arm template or configuration files ",
"in the container env or cmd sections will be printed out with this flag.",
"These are outputed secrets that you must protect. Be sure that you do not include these secrets in your",
"source control. Also verify that no secrets are present in the logs of your command or script.",
"For additional information, see http://aka.ms/clisecrets. \n",
)

if print_existing_policy:
print_existing_policy_from_arm_template(arm_template, arm_template_parameters)
sys.exit(0)
Expand Down