Skip to content

Maintenance: Create a e2e test for multiple AppConfig profile calls #3200

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

Closed
1 of 2 tasks
leandrodamascena opened this issue Oct 13, 2023 · 3 comments
Closed
1 of 2 tasks
Labels
help wanted Could use a second pair of eyes/hands internal Maintenance changes

Comments

@leandrodamascena
Copy link
Contributor

Why is this needed?

When using the AppConfig Provider to fetch user specific configurations at runtime based on the identity of the requester that invokes our lambda function, we expect the correct configuration to be returned.

What we are seeing is that when multiple requests are being made at the same time from different users (across different lambda invocations), the AppConfig provider will be called to fetch the configuration with the right username, but return the wrong configuration profile (it will be called with user A, but return user B's profile). We have tried turning off caching of the profiles and this did not help. All we are doing is calling that initialized AppConfigProvider with the config_name that is specific to each user when we need to fetch that profile. It is important to note that for each request, we only need to fetch one profile that is scoped to the user who made the request.

Refer: #3194

Which area does this relate to?

Tests

Solution

No response

Acknowledgment

@leandrodamascena leandrodamascena added triage Pending triage from maintainers internal Maintenance changes labels Oct 13, 2023
@leandrodamascena leandrodamascena removed the triage Pending triage from maintainers label Oct 15, 2023
@heitorlessa heitorlessa added the triage Pending triage from maintainers label Jun 9, 2024
@heitorlessa
Copy link
Contributor

@leandrodamascena do we still need it?

@heitorlessa heitorlessa added help wanted Could use a second pair of eyes/hands and removed triage Pending triage from maintainers labels Jun 11, 2024
@leandrodamascena
Copy link
Contributor Author

We don't need this anymore! We didn't have any bug report regarding this.

Copy link
Contributor

github-actions bot commented Aug 6, 2024

⚠️COMMENT VISIBILITY WARNING⚠️

This issue is now closed. Please be mindful that future comments are hard for our team to see.

If you need more assistance, please either tag a team member or open a new issue that references this one.

If you wish to keep having a conversation with other community members under this issue feel free to do so.

@leandrodamascena leandrodamascena moved this from Coming soon to Closed in Powertools for AWS Lambda (Python) Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Could use a second pair of eyes/hands internal Maintenance changes
Projects
Development

No branches or pull requests

2 participants