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

feat: sync env vars for Gateway #90

Merged
merged 2 commits into from
Apr 17, 2024
Merged

Conversation

JoeCap08055
Copy link
Contributor

Description

  • Remove unused env vars
  • Honor API_PORT
  • Rename ENVIRONMENT to CHAIN_ENVIRONMENT
  • Make ConfigService getters consistent with other services
  • Add doc comments to env.template
  • Add ENVIRONMENT.md
  • Update README

Closes #86

- Remove unused env vars
- Honor API_PORT
- Rename ENVIRONMENT to CHAIN_ENVIRONMENT
- Make ConfigService getters consistent with other services
- Add doc comments to env.template
- Add ENVIRONMENT.md
- Update README
Copy link
Collaborator

@aramikm aramikm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good. Just a comment about consistency.

PROVIDER_ACCOUNT_SEED_PHRASE: string;
CAPACITY_LIMIT: ICapacityLimit;
FILE_UPLOAD_MAX_SIZE_IN_BYTES: number;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Even though this is not being used directly from config I think it would help to keep it around for future and consistency between the env variables and config fields . This configuration has being used directly inside ApiController since I couldn't find an easy way to inject the config service inside there.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed--I restored this to the environment, and found a way to honor it from the ConfigService using async default module options

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, I would lower this value for test and check to ensure it does limit the uploaded file and works as expected

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I did that--errored as expected.

libs/common/src/config/env.config.ts Show resolved Hide resolved
@JoeCap08055 JoeCap08055 merged commit 5132857 into main Apr 17, 2024
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sync env vars for gateway
2 participants