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

Commit

Permalink
fix: container environment variables (#1202)
Browse files Browse the repository at this point in the history
* fix: container environment names

* sync

---------

Co-authored-by: Frank <[email protected]>
  • Loading branch information
sellooh and fwang authored Oct 16, 2024
1 parent 8a40f6f commit e442978
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions platform/src/components/aws/service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -609,9 +609,8 @@ export class Service extends Component implements Link.Linkable {
containerDefinitions: $jsonStringify(
all([
containers,
args.environment ?? [],
Link.propertiesToEnv(Link.getProperties(args.link)),
]).apply(([containers, env, linkEnvs]) =>
]).apply(([containers, linkEnvs]) =>
containers.map((container) => {
return {
name: container.name,
Expand All @@ -628,9 +627,10 @@ export class Service extends Component implements Link.Linkable {
"awslogs-stream-prefix": "/service",
},
},
environment: Object.entries({ ...env, ...linkEnvs }).map(
([name, value]) => ({ name, value }),
),
environment: Object.entries({
...container.environment,
...linkEnvs,
}).map(([name, value]) => ({ name, value })),
linuxParameters: {
initProcessEnabled: true,
},
Expand Down

0 comments on commit e442978

Please sign in to comment.