Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use powertools for fetching secret #28

Open
deeheber opened this issue Oct 27, 2024 · 1 comment
Open

Use powertools for fetching secret #28

deeheber opened this issue Oct 27, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@deeheber
Copy link
Owner

The checkCurrentWeather function is currently using the AWS secrets manager lambda extension which is fine, but powertools released what appears to be a simpler abstraction (possibly on top of this extension).

Take a look into replacing the explicit use the lambda extension with powertools. https://docs.powertools.aws.dev/lambda/typescript/latest/utilities/parameters/

@deeheber deeheber added the enhancement New feature or request label Oct 27, 2024
@deeheber
Copy link
Owner Author

We should probably look into doing #18 first because if this Lambda can be fully replaced with an http state then this issue won't be needed.

It's also possible that after researching how power tools works, we find that the current way of doing it is preferred. Would ideally like the abstraction that is most maintainable + easily readable in the code. Lambda powertools seems like a stable and supported library by AWS at the time of writing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant