Skip to content

Commit 72599a7

Browse files
fix: incorrectly setting secured value
1 parent 61da877 commit 72599a7

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

helpers/logging.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
import datetime
88

99
class Logger:
10+
"""Build Logger"""
1011
def __init__(self, enable_log_file=False, log_level='INFO'):
1112
self.log_name = os.path.basename(inspect.stack()[1].filename)
1213
self.log_level = log_level
@@ -21,7 +22,8 @@ def __logging(self):
2122
formatter = logging.Formatter(fmt="[%(asctime)s] - [%(levelname)s] - [%(name)s.%(funcName)s:%(lineno)d] - [%(message)s]")
2223
file_handler = logging.FileHandler(
2324
filename=f"{self.log_name}-{datestamp}.log",
24-
mode="a")
25+
mode="a"
26+
)
2527
file_handler.setFormatter(formatter)
2628
logger.addHandler(file_handler)
2729

manage-bitbucket-env.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ def export_variables(workspace, repo_slug, deployment_name, output_file, auth):
131131
export_vars = []
132132
for var in variables:
133133
if not var["secured"]:
134-
export_vars.append({"key": var["key"], "value": var["value"], "secured": False})
134+
export_vars.append({"key": var["key"], "value": var["value"], "secured": var["secured"]})
135135
else:
136136
log.info("Secured variable '%s' will not be exported. Use --export-secret-keys for a list of secure keys.", var['key'])
137137
log.debug("Preparing to export %s variables", len(export_vars))
@@ -150,9 +150,9 @@ def export_all_variables(workspace, repo_slug, deployment_name, output_file, aut
150150
export_vars = []
151151
for var in variables:
152152
if var["secured"]:
153-
export_vars.append({"key": var["key"], "value": "", "secured": True})
153+
export_vars.append({"key": var["key"], "value": "", "secured": var["secured"]})
154154
else:
155-
export_vars.append({"key": var["key"], "value": var["value"], "secured": False})
155+
export_vars.append({"key": var["key"], "value": var["value"], "secured": var["secured"]})
156156
log.debug("Preparing to export %s variables", len(export_vars))
157157
with open(output_file, "w", encoding="utf-8") as f:
158158
json.dump(export_vars, f, indent=4)
@@ -181,7 +181,7 @@ def update_vars(workspace, repo_slug, environment_uuid, existing_vars, var, auth
181181
payload = {
182182
"key": var["key"],
183183
"value": var["value"],
184-
"secured": False,
184+
"secured": var["secured"],
185185
"environment": {
186186
"type": "deployment_environment",
187187
"uuid": environment_uuid
@@ -196,7 +196,7 @@ def update_vars(workspace, repo_slug, environment_uuid, existing_vars, var, auth
196196
payload = {
197197
"key": var["key"],
198198
"value": var["value"],
199-
"secured": False,
199+
"secured": var["secured"],
200200
"environment": {
201201
"type": "deployment_environment",
202202
"uuid": environment_uuid

0 commit comments

Comments
 (0)