Skip to content
This repository has been archived by the owner on Oct 13, 2023. It is now read-only.

Commit

Permalink
fix snyk errors
Browse files Browse the repository at this point in the history
  • Loading branch information
ashwindasr authored and sosiouxme committed Apr 4, 2023
1 parent e4b230d commit cc6ceb4
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 5 deletions.
2 changes: 1 addition & 1 deletion doozerlib/constants.py
Expand Up @@ -15,7 +15,7 @@
DB_HOST = "DOOZER_DB_HOST"
DB_PORT = "DOOZER_DB_PORT"
DB_USER = "DOOZER_DB_USER"
DB_PWD = "DOOZER_DB_PASSWORD"
DB_PWD_NAME = "DOOZER_DB_PASSWORD"
DB_NAME = "DOOZER_DB_NAME"

# default db parameters
Expand Down
6 changes: 3 additions & 3 deletions doozerlib/dblib.py
Expand Up @@ -52,7 +52,7 @@ def wrapper(*args, **kwargs):
db.connection = mysql_connector.connect(
host=os.getenv(constants.DB_HOST, constants.default_db_params[constants.DB_HOST]),
user=os.getenv(constants.DB_USER),
password=os.getenv(constants.DB_PWD),
password=os.getenv(constants.DB_PWD_NAME),
database=os.getenv(constants.DB_NAME, constants.default_db_params[constants.DB_NAME]))
break
except Exception as e:
Expand Down Expand Up @@ -105,7 +105,7 @@ def check_missing_db_env_var(self):
and returns True.
"""

if not (constants.DB_USER in os.environ and constants.DB_PWD in os.environ):
if not (constants.DB_USER in os.environ and constants.DB_PWD_NAME in os.environ):
self.runtime.logger.info("Environment variables required for db operation missing. Doozer will be running"
"in no DB use mode.")
return False
Expand All @@ -116,7 +116,7 @@ def check_missing_db_env_var(self):
self.host = os.getenv(constants.DB_HOST, constants.default_db_params[constants.DB_HOST])
self.port = os.getenv(constants.DB_PORT, constants.default_db_params[constants.DB_PORT])
self.db = os.getenv(constants.DB_NAME, constants.default_db_params[constants.DB_NAME])
self.pwd = os.getenv(constants.DB_PWD)
self.pwd = os.getenv(constants.DB_PWD_NAME)
self.db_user = os.getenv(constants.DB_USER)

self.runtime.logger.info("Found all environment variables required for db setup.")
Expand Down
2 changes: 1 addition & 1 deletion doozerlib/metadata.py
Expand Up @@ -8,7 +8,7 @@
from collections import OrderedDict
from enum import Enum
from typing import Any, Dict, List, NamedTuple, Optional, Tuple, Union
from xml.etree import ElementTree
from defusedxml import ElementTree

import dateutil.parser
import requests
Expand Down
1 change: 1 addition & 0 deletions requirements.txt
Expand Up @@ -2,6 +2,7 @@ aiofiles
bashlex
click >= 8.1.3
dockerfile-parse >= 0.0.13
defusedxml
future
koji
PyGitHub >= 1.46
Expand Down

0 comments on commit cc6ceb4

Please sign in to comment.