Skip to content

Commit 306253c

Browse files
committed
Enhances Docker deployment with secrets usage
Replaces environment variables with secrets for secure and consistent deployment Strengthens best practices by centralizing configuration references
1 parent d8c9822 commit 306253c

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

.github/workflows/docker-deploy.yml

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -51,22 +51,20 @@ jobs:
5151
5252
deploy-to-appservice:
5353
needs: build-and-push
54-
env:
55-
EDITOR_APP_NAME: ${{ secrets.EDITOR_APP_NAME }}
56-
RESOURCE_GROUP: ${{ secrets.RESOURCE_GROUP }}
57-
EDITOR_API_APP_NAME: ${{ secrets.EDITOR_API_APP_NAME }}
58-
if: ${{ env.EDITOR_APP_NAME != '' && env.RESOURCE_GROUP != '' && env.EDITOR_API_APP_NAME != '' }}
5954
runs-on: ubuntu-latest
6055
steps:
6156
- name: Azure Login
57+
if: ${{ secrets.EDITOR_APP_NAME != '' && secrets.RESOURCE_GROUP != '' && secrets.EDITOR_API_APP_NAME != '' }}
6258
uses: azure/login@v1
6359
with:
6460
creds: ${{ secrets.AZURE_CREDENTIALS }}
6561

6662
- name: Deploy editor to App Service (container)
63+
if: ${{ secrets.EDITOR_APP_NAME != '' && secrets.RESOURCE_GROUP != '' && secrets.EDITOR_API_APP_NAME != '' }}
6764
run: |
68-
az webapp config container set --name ${{ env.EDITOR_APP_NAME }} --resource-group ${{ env.RESOURCE_GROUP }} --docker-custom-image-name ${{ needs.build-and-push.outputs.registry }}/app-scripting-editor:latest
65+
az webapp config container set --name ${{ secrets.EDITOR_APP_NAME }} --resource-group ${{ secrets.RESOURCE_GROUP }} --docker-custom-image-name ${{ needs.build-and-push.outputs.registry }}/app-scripting-editor:latest
6966
7067
- name: Deploy editor-api to App Service (container)
68+
if: ${{ secrets.EDITOR_APP_NAME != '' && secrets.RESOURCE_GROUP != '' && secrets.EDITOR_API_APP_NAME != '' }}
7169
run: |
72-
az webapp config container set --name ${{ env.EDITOR_API_APP_NAME }} --resource-group ${{ env.RESOURCE_GROUP }} --docker-custom-image-name ${{ needs.build-and-push.outputs.registry }}/app-scripting-editor-api:latest
70+
az webapp config container set --name ${{ secrets.EDITOR_API_APP_NAME }} --resource-group ${{ secrets.RESOURCE_GROUP }} --docker-custom-image-name ${{ needs.build-and-push.outputs.registry }}/app-scripting-editor-api:latest

0 commit comments

Comments
 (0)