Skip to content

Commit 2b58c24

Browse files
committed
Replace all inputs by env vars. They were duplicated
1 parent 23af033 commit 2b58c24

File tree

3 files changed

+45
-28
lines changed

3 files changed

+45
-28
lines changed

.github/workflows/firebase-app-distribution.yml

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -43,22 +43,25 @@ jobs:
4343
XCODE_AUTHENTICATION_KEY_ID: ${{ secrets.XCODE_AUTHENTICATION_KEY_ID }}
4444
XCODE_AUTHENTICATION_KEY_ISSUER_ID: ${{ secrets.XCODE_AUTHENTICATION_KEY_ISSUER_ID }}
4545
XCODE_AUTHENTICATION_KEY_BASE64: ${{ secrets.XCODE_AUTHENTICATION_KEY_BASE64 }}
46+
47+
# Firebase env
4648
FIREBASE_SERVICE_ACCOUNT_JSON: ${{ secrets.FIREBASE_SERVICE_ACCOUNT_JSON }}
4749
FIREBASE_APP_ID: ${{ secrets.FIREBASE_APP_ID }}
50+
FIREBASE_RELEASE_NAME: ${{ env.RELEASE_NAME }}
4851

49-
# Build configuration
52+
# App build configuration
5053
MERCHANT_CLIENT_KEY: ${{ secrets.DEMO_APP_TEST_ENV_CLIENT_KEY }}
5154
MERCHANT_SERVER_HOST: ${{ secrets.MERCHANT_SERVER_HOST }}
5255
MERCHANT_ACCOUNT: ${{ secrets.MERCHANT_ACCOUNT }}
5356
ADYEN_SERVER_API_KEY: ${{ secrets.DEMO_SERVER_TEST_ENV_API_KEY }}
5457
APPLE_TEAM_IDENTIFIER: ${{ secrets.APPLE_DEVELOPMENT_TEAM_ID }}
5558
ENVIRONMENT: ${{ secrets.ENVIRONMENT }}
5659
run: |
60+
AUTH_KEY_PATH=$RUNNER_TEMP/authentication_key_path.p8
61+
rm -f $AUTH_KEY_PATH
62+
echo -n "$XCODE_AUTHENTICATION_KEY_BASE64" | base64 --decode -o $AUTH_KEY_PATH
63+
64+
echo "AUTH_KEY_PATH=$AUTH_KEY_PATH" >> $GITHUB_ENV
65+
5766
chmod +x ./Scripts/publish-demo-app.sh
58-
./Scripts/publish-demo-app.sh \
59-
"${{ secrets.APPLE_ID_USERNAME }}" \
60-
"${{ secrets.APPLE_APP_SPECIFIC_PASSWORD }}" \
61-
"${{ secrets.XCODE_AUTHENTICATION_KEY_PATH }}" \
62-
"firebase"
63-
env:
64-
FIREBASE_RELEASE_NAME: ${{ env.RELEASE_NAME }}
67+
./Scripts/publish-demo-app.sh "firebase"

.github/workflows/publish-demo-app.yml

Lines changed: 19 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -108,21 +108,30 @@ jobs:
108108
sed -i '' "s/static let componentsEnvironment.*=.*/static let componentsEnvironment = Environment.$ENVIRONMENT/g" ./Demo/Configuration.swift
109109
- name: 🚀 Publish Demo App
110110
env:
111+
# Workflow configuration
111112
APPLE_ID_USERNAME: ${{ secrets.APPLE_ID_USERNAME }}
112113
APPLE_APP_SPECIFIC_PASSWORD: ${{ secrets.APPLE_APP_SPECIFIC_PASSWORD }}
114+
APPLE_PAY_MERCHANT_IDENTIFIER: ${{ secrets.APPLE_PAY_MERCHANT_IDENTIFIER }}
113115
XCODE_AUTHENTICATION_KEY_ID: ${{ secrets.XCODE_AUTHENTICATION_KEY_ID }}
114116
XCODE_AUTHENTICATION_KEY_ISSUER_ID: ${{ secrets.XCODE_AUTHENTICATION_KEY_ISSUER_ID }}
115117
XCODE_AUTHENTICATION_KEY_BASE64: ${{ secrets.XCODE_AUTHENTICATION_KEY_BASE64 }}
116-
117-
CLIENT_KEY: ${{ secrets.DEMO_APP_TEST_ENV_CLIENT_KEY }}
118-
DEMO_SERVER_API_KEY: ${{ secrets.DEMO_SERVER_TEST_ENV_API_KEY }}
118+
FIREBASE_SERVICE_ACCOUNT_JSON: ${{ secrets.FIREBASE_SERVICE_ACCOUNT_JSON }}
119+
FIREBASE_APP_ID: ${{ secrets.FIREBASE_APP_ID }}
120+
FIREBASE_RELEASE_NAME: ${{ env.RELEASE_NAME }}
121+
122+
# Build configuration
123+
MERCHANT_CLIENT_KEY: ${{ secrets.DEMO_APP_TEST_ENV_CLIENT_KEY }}
124+
MERCHANT_SERVER_HOST: ${{ secrets.MERCHANT_SERVER_HOST }}
119125
MERCHANT_ACCOUNT: ${{ secrets.MERCHANT_ACCOUNT }}
120-
APPLE_DEVELOPMENT_TEAM_ID: ${{ secrets.APPLE_DEVELOPMENT_TEAM_ID }}
126+
ADYEN_SERVER_API_KEY: ${{ secrets.DEMO_SERVER_TEST_ENV_API_KEY }}
127+
APPLE_TEAM_IDENTIFIER: ${{ secrets.APPLE_DEVELOPMENT_TEAM_ID }}
121128
ENVIRONMENT: ${{ secrets.ENVIRONMENT }}
122-
APPLE_PAY_MERCHANT_IDENTIFIER: ${{ secrets.APPLE_PAY_MERCHANT_IDENTIFIER }}
123-
run: |
124-
AUTH_KEY_PATH=$RUNNER_TEMP/authentication_key_path.p8
125-
rm -f $AUTH_KEY_PATH
126-
echo -n "$XCODE_AUTHENTICATION_KEY_BASE64" | base64 --decode -o $AUTH_KEY_PATH
129+
run: |
130+
AUTH_KEY_PATH=$RUNNER_TEMP/authentication_key_path.p8
131+
rm -f $AUTH_KEY_PATH
132+
echo -n "$XCODE_AUTHENTICATION_KEY_BASE64" | base64 --decode -o $AUTH_KEY_PATH
133+
134+
echo "AUTH_KEY_PATH=$AUTH_KEY_PATH" >> $GITHUB_ENV
127135
128-
./Scripts/publish-demo-app.sh "$APPLE_ID_USERNAME" "$APPLE_APP_SPECIFIC_PASSWORD" "$AUTH_KEY_PATH"
136+
chmod +x ./Scripts/publish-demo-app.sh
137+
./Scripts/publish-demo-app.sh

Scripts/publish-demo-app.sh

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -9,14 +9,11 @@ IPA_PATH="$BUILD_PATH/AdyenUIHost.ipa"
99
EXPORT_OPTIONS_PLIST="$SCRIPT_DIR/exportOptions.plist"
1010

1111
# Input arguments
12-
APPLE_ID_USERNAME="$1"
13-
APPLE_APP_SPECIFIC_PASSWORD="$2"
14-
AUTH_KEY_PATH="$3"
15-
DISTRIBUTION_TYPE="${4:-testflight}" # testflight (default) or firebase
12+
DISTRIBUTION_TYPE="${1:-testflight}" # testflight (default) or firebase
1613

1714
# Validate inputs
18-
if [[ -z "$APPLE_ID_USERNAME" || -z "$APPLE_APP_SPECIFIC_PASSWORD" || -z "$AUTH_KEY_PATH" ]]; then
19-
echo "❌ Usage: $0 <APPLE_ID_USERNAME> <APPLE_APP_SPECIFIC_PASSWORD> <AUTH_KEY_PATH> [distribution_type]"
15+
if [[ -z "$DISTRIBUTION_TYPE"]]; then
16+
echo "❌ Usage: $0 [distribution_type]"
2017
exit 1
2118
fi
2219

@@ -32,8 +29,7 @@ fi
3229
: "${XCODE_AUTHENTICATION_KEY_ID:?Environment variable XCODE_AUTHENTICATION_KEY_ID not set}"
3330
: "${XCODE_AUTHENTICATION_KEY_ISSUER_ID:?Environment variable XCODE_AUTHENTICATION_KEY_ISSUER_ID not set}"
3431
: "${XCODE_AUTHENTICATION_KEY_BASE64:?Environment variable XCODE_AUTHENTICATION_KEY_BASE64 not set}"
35-
: "${FIREBASE_SERVICE_ACCOUNT_JSON:?Environment variable FIREBASE_SERVICE_ACCOUNT_JSON not set}"
36-
: "${FIREBASE_APP_ID:?Environment variable FIREBASE_APP_ID not set}"
32+
: "${AUTH_KEY_PATH:?Environment variable AUTH_KEY_PATH not set}"
3733

3834
: "${MERCHANT_CLIENT_KEY:?Environment variable MERCHANT_CLIENT_KEY not set}"
3935
: "${MERCHANT_SERVER_HOST:?Environment variable MERCHANT_SERVER_HOST not set}"
@@ -42,6 +38,15 @@ fi
4238
: "${APPLE_TEAM_IDENTIFIER:?Environment variable APPLE_TEAM_IDENTIFIER not set}"
4339
: "${ENVIRONMENT:?Environment variable ENVIRONMENT not set}"
4440

41+
if [[ "$DISTRIBUTION" == "firebase" ]]; then
42+
: "${FIREBASE_SERVICE_ACCOUNT_JSON:?Environment variable FIREBASE_SERVICE_ACCOUNT_JSON not set}"
43+
: "${FIREBASE_APP_ID:?Environment variable FIREBASE_APP_ID not set}"
44+
: "${FIREBASE_RELEASE_NAME:?Environment variable FIREBASE_RELEASE_NAME not set}"
45+
46+
echo "ℹ️ Firebase distribution selected — Firebase env vars validated."
47+
fi
48+
49+
4550
echo "🧹 Cleaning project..."
4651
xcodebuild clean -project Adyen.xcodeproj \
4752
-scheme AdyenUIHost \
@@ -97,7 +102,7 @@ if [[ "$DISTRIBUTION_TYPE" == "testflight" ]]; then
97102
echo "✅ TestFlight upload complete!"
98103

99104
elif [[ "$DISTRIBUTION_TYPE" == "firebase" ]]; then
100-
echo "☁️ Uploading to Firebase App Distribution..."
105+
echo "🔥 Uploading to Firebase App Distribution..."
101106

102107
if [[ -z "${FIREBASE_SERVICE_ACCOUNT_JSON:-}" || -z "${FIREBASE_APP_ID:-}" ]]; then
103108
echo "❌ FIREBASE_SERVICE_ACCOUNT_JSON and FIREBASE_APP_ID environment variables must be set for Firebase distribution"
@@ -112,7 +117,7 @@ elif [[ "$DISTRIBUTION_TYPE" == "firebase" ]]; then
112117
firebase appdistribution:distribute "$IPA_PATH" \
113118
--app "$FIREBASE_APP_ID" \
114119
--groups "ios-team" \
115-
--release-notes "Build from branch ${GITHUB_REF:-manual}" \
120+
--release-notes "${FIREBASE_RELEASE_NAME:-Build from branch ${GITHUB_REF_NAME:-manual}}" \
116121
--service-account "$FIREBASE_JSON_PATH"
117122

118123
echo "✅ Firebase upload complete!"

0 commit comments

Comments
 (0)