Skip to content

Commit e680d92

Browse files
committed
Try to remove any secrets in envs during build
1 parent d8cdef4 commit e680d92

File tree

2 files changed

+29
-66
lines changed

2 files changed

+29
-66
lines changed

docker-compose.build.yml

Lines changed: 29 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@ services:
88
- type=registry,ref=${WS_DOCKER_REGISTRY}/workspaces-frontend:buildcache
99
cache_to:
1010
- type=registry,ref=${WS_DOCKER_REGISTRY}/workspaces-frontend:buildcache,mode=max
11+
12+
# these get baked into the JavaScript at build time--so ARGS not ENVS
1113
args:
1214
VITE_TDEI_API_URL: ${WS_TDEI_API_URL}
1315
VITE_TDEI_USER_API_URL: ${WS_TDEI_BACKEND_URL}
@@ -84,30 +86,32 @@ services:
8486
# OSM website/OSM API
8587
osm-rails:
8688
image: ${WS_DOCKER_REGISTRY}/workspaces-osm-rails:${ENV}
87-
environment:
88-
RAILS_ENV: production
89-
SECRET_KEY_BASE: ${WS_OSM_SECRET_KEY_BASE}
90-
WS_OSM_HOST: ${WS_OSM_HOST}
91-
WS_OSM_DB_HOST: ${WS_OSM_DB_HOST}
92-
WS_OSM_DB_USER: ${WS_OSM_DB_USER}
93-
WS_OSM_DB_PASS: ${WS_OSM_DB_PASS}
94-
WS_OSM_DB_NAME: ${WS_OSM_DB_NAME}
95-
WS_MAIL_NAME: ${WS_MAIL_NAME}
96-
WS_MAIL_FROM: ${WS_MAIL_FROM}
97-
WS_MAIL_CONTACT: ${WS_MAIL_CONTACT}
98-
WS_MAIL_RETURN_PATH: ${WS_MAIL_RETURN_PATH}
99-
WS_SMTP_DOMAIN: ${WS_SMTP_DOMAIN}
100-
WS_SMTP_HOST: ${WS_SMTP_HOST}
101-
WS_SMTP_PORT: ${WS_SMTP_PORT}
102-
WS_SMTP_USER: ${WS_SMTP_USER}
103-
WS_SMTP_PASS: ${WS_SMTP_PASS}
104-
PIDFILE: /tmp/pids/server.pid
89+
# environment:
90+
# RAILS_ENV: production
91+
# SECRET_KEY_BASE: ${WS_OSM_SECRET_KEY_BASE}
92+
# WS_OSM_HOST: ${WS_OSM_HOST}
93+
# WS_OSM_DB_HOST: ${WS_OSM_DB_HOST}
94+
# WS_OSM_DB_USER: ${WS_OSM_DB_USER}
95+
# WS_OSM_DB_PASS: ${WS_OSM_DB_PASS}
96+
# WS_OSM_DB_NAME: ${WS_OSM_DB_NAME}
97+
# WS_MAIL_NAME: ${WS_MAIL_NAME}
98+
# WS_MAIL_FROM: ${WS_MAIL_FROM}
99+
# WS_MAIL_CONTACT: ${WS_MAIL_CONTACT}
100+
# WS_MAIL_RETURN_PATH: ${WS_MAIL_RETURN_PATH}
101+
# WS_SMTP_DOMAIN: ${WS_SMTP_DOMAIN}
102+
# WS_SMTP_HOST: ${WS_SMTP_HOST}
103+
# WS_SMTP_PORT: ${WS_SMTP_PORT}
104+
# WS_SMTP_USER: ${WS_SMTP_USER}
105+
# WS_SMTP_PASS: ${WS_SMTP_PASS}
106+
# PIDFILE: /tmp/pids/server.pid
107+
105108
build:
106-
args:
107-
WS_OSM_DB_HOST: ${WS_OSM_DB_HOST}
108-
WS_OSM_DB_USER: ${WS_OSM_DB_USER}
109-
WS_OSM_DB_PASS: ${WS_OSM_DB_PASS}
110-
WS_OSM_DB_NAME: ${WS_OSM_DB_NAME}
109+
# args:
110+
# WS_OSM_DB_HOST: ${WS_OSM_DB_HOST}
111+
# WS_OSM_DB_USER: ${WS_OSM_DB_USER}
112+
# WS_OSM_DB_PASS: ${WS_OSM_DB_PASS}
113+
# WS_OSM_DB_NAME: ${WS_OSM_DB_NAME}
114+
111115
cache_from:
112116
- type=registry,ref=${WS_DOCKER_REGISTRY}/workspaces-osm-rails:buildcache
113117
cache_to:
@@ -134,25 +138,6 @@ services:
134138
extends: osm-cgimap
135139
image: ${WS_DOCKER_REGISTRY}/workspaces-osm-cgimap:${CODE_VERSION}
136140

137-
# tasking manager
138-
# tasks-frontend:
139-
# image: ${WS_DOCKER_REGISTRY}/workspaces-tasks-frontend:${ENV}
140-
# build:
141-
# cache_from:
142-
# - type=registry,ref=${WS_DOCKER_REGISTRY}/workspaces-tasks-frontend:buildcache
143-
# cache_to:
144-
# - type=registry,ref=${WS_DOCKER_REGISTRY}/workspaces-tasks-frontend:buildcache,mode=max
145-
# context: tasking-manager
146-
# dockerfile: ./scripts/docker/Dockerfile.frontend
147-
# args:
148-
# TM_APP_API_URL: ${WS_TASKS_URL}
149-
# #TM_CONSUMER_KEY
150-
# #TM_CONSUMER_SECRET
151-
152-
# tasks-frontend_tag:
153-
# extends: tasks-frontend
154-
# image: ${WS_DOCKER_REGISTRY}/workspaces-tasks-frontend:${CODE_VERSION}
155-
156141
# tasking manager
157142
tasks-backend:
158143
image: ${WS_DOCKER_REGISTRY}/workspaces-tasks-backend:${ENV}
@@ -177,6 +162,8 @@ services:
177162
- type=registry,ref=${WS_DOCKER_REGISTRY}/workspaces-leaderboard-frontend:buildcache,mode=max
178163
context: ./leaderboard/frontend
179164
dockerfile: Dockerfile
165+
166+
# these get baked into the JavaScript at build time--so ARGS not ENVS
180167
args:
181168
VITE_TDEI_API_URL: ${WS_TDEI_API_URL}
182169
VITE_TDEI_USER_API_URL: ${WS_TDEI_BACKEND_URL}

docker-compose.deploy.yml

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -129,19 +129,6 @@ services:
129129
- traefik.http.routers.pathways-editor.entrypoints=websecure
130130
- traefik.http.routers.pathways-editor.tls.certresolver=myresolver
131131

132-
# for debugging only
133-
#
134-
#osm-log-proxy:
135-
# image: ${WS_DOCKER_REGISTRY}/workspaces-osm-log-proxy:${WS_DOCKER_TAG}
136-
# depends_on:
137-
# - osm-web
138-
# #labels:
139-
# # - traefik.enable=true
140-
# # - traefik.http.routers.osm-log-proxy.rule=Host(`${WS_OSM_HOST}`)
141-
# # - traefik.http.services.osm-log-proxy.loadbalancer.server.port=80
142-
# # - traefik.http.routers.osm-log-proxy.entrypoints=websecure
143-
# # - traefik.http.routers.osm-log-proxy.tls.certresolver=myresolver
144-
145132
# workspaces APIs (OSM Rails and C-accelerated)
146133
osm-web:
147134
image: ${WS_DOCKER_REGISTRY}/workspaces-osm-web:${WS_DOCKER_TAG}
@@ -178,17 +165,6 @@ services:
178165
CGIMAP_MAP_NODES: ${WS_OSM_MAX_EXPORT_NODES}
179166
CGIMAP_MAP_AREA: ${WS_OSM_MAX_EXPORT_AREA}
180167

181-
# tasking manager (*not* Workspaces)
182-
# tasks-frontend:
183-
# image: ${WS_DOCKER_REGISTRY}/workspaces-tasks-frontend:${WS_DOCKER_TAG}
184-
# restart: always
185-
# labels:
186-
# - traefik.enable=true
187-
# - traefik.http.routers.tasks-frontend.rule=Host(`${WS_TASKS_HOST}`)
188-
# - traefik.http.services.tasks-frontend.loadbalancer.server.port=80
189-
# - traefik.http.routers.tasks-frontend.entrypoints=websecure
190-
# - traefik.http.routers.tasks-frontend.tls.certresolver=myresolver
191-
192168
tasks-backend:
193169
image: ${WS_DOCKER_REGISTRY}/workspaces-tasks-backend:${WS_DOCKER_TAG}
194170
restart: always

0 commit comments

Comments
 (0)