Skip to content

Commit 99afb2f

Browse files
Fix Img environment variable retrieval in docker-build and update React dependencies in package.json
1 parent 9fc5aab commit 99afb2f

2 files changed

Lines changed: 12 additions & 9 deletions

File tree

infra/scripts/docker-build.sh

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,12 @@ get_azd_env_value_or_default() {
1414
local default="$2"
1515
local required="${3:-false}"
1616

17-
value=$(azd env get-value "$key" 2>/dev/null || echo "")
17+
local value
18+
value=$(azd env get-value "$key" 2>/dev/null)
19+
local exit_code=$?
1820

19-
if [ -z "$value" ]; then
20-
if [ "$required" = true ]; then
21+
if [ $exit_code -ne 0 ] || [ -z "$value" ]; then
22+
if [ "$required" = "true" ]; then
2123
echo "❌ Required environment key '$key' not found." >&2
2224
exit 1
2325
else
@@ -27,6 +29,7 @@ get_azd_env_value_or_default() {
2729

2830
echo "$value"
2931
}
32+
3033
# Required env variables
3134
AZURE_SUBSCRIPTION_ID=$(get_azd_env_value_or_default "AZURE_SUBSCRIPTION_ID" "" true)
3235
ENV_NAME=$(get_azd_env_value_or_default "AZURE_ENV_NAME" "" true)
@@ -37,8 +40,8 @@ AZURE_ENV_IMAGETAG=$(get_azd_env_value_or_default "AZURE_ENV_IMAGETAG" "latest"
3740
CONTAINER_WEB_APP_NAME=$(get_azd_env_value_or_default "CONTAINER_WEB_APP_NAME" "" true)
3841
CONTAINER_API_APP_NAME=$(get_azd_env_value_or_default "CONTAINER_API_APP_NAME" "" true)
3942
CONTAINER_APP_NAME=$(get_azd_env_value_or_default "CONTAINER_APP_NAME" "" true)
40-
$ACR_NAME = $(get_azd_env_value_or_default "CONTAINER_REGISTRY_NAME" "" true)
41-
$ACR_ENDPOINT = $(get_azd_env_value_or_default "CONTAINER_REGISTRY_LOGIN_SERVER" "" true)
43+
ACR_NAME=$(get_azd_env_value_or_default "CONTAINER_REGISTRY_NAME" "" true)
44+
ACR_ENDPOINT=$(get_azd_env_value_or_default "CONTAINER_REGISTRY_LOGIN_SERVER" "" true)
4245

4346
echo "Using the following parameters:"
4447
echo "AZURE_SUBSCRIPTION_ID = $AZURE_SUBSCRIPTION_ID"

src/ContentProcessorWeb/package.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,8 @@
1616
"nth-check": "2.1.1",
1717
"postcss": "8.5.6",
1818
"prismjs": "^1.30.0",
19-
"react": "^19.1.0",
20-
"react-dom": "^19.1.0",
19+
"react": "^18.3.1",
20+
"react-dom": "^18.3.1",
2121
"react-dropzone": "^14.3.8",
2222
"react-medium-image-zoom": "^5.2.14",
2323
"react-redux": "^9.1.2",
@@ -51,8 +51,8 @@
5151
},
5252
"devDependencies": {
5353
"@eslint/js": "^9.30.0",
54-
"@types/react": "^19.1.8",
55-
"@types/react-dom": "^19.1.6",
54+
"@types/react": "^18.3.18",
55+
"@types/react-dom": "^18.3.5",
5656
"@types/react-router-dom": "^5.3.3",
5757
"@types/react-virtualized": "^9.22.2",
5858
"@types/react-window": "^1.8.8",

0 commit comments

Comments
 (0)