[identity] config for LocalStack
ClosedPublic
Actions

Authored by varun on Mar 30 2023, 12:32 PM.

Details

Reviewers

• jon
bartek

Commits

rCOMM86ee2981a2e3: [identity] config for LocalStack

Summary

we need to introduce some more config for our DDB client to be able to connect to the LocalStack endpoint.

If the LOCALSTACK_ENDPOINT env var is set, the DDB client will connect to that endpoint instead of the AWS prod endpoint.

Test Plan

called GenerateNonce with the env var set, used AWS cli to confirm that the nonce was added to my LocalStack DDB identity-nonces table
called with the env var unset, and the nonce was added to my actual DDB test table

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Mar 30 2023, 12:32 PM

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptMar 30 2023, 12:32 PM

varun added a parent revision: D7256: [identity] bump aws-config and aws-sdk-dynamodb crate versions.Mar 30 2023, 12:33 PM

Harbormaster completed remote builds in B17843: Diff 24439.Mar 30 2023, 12:48 PM

varun requested review of this revision.Mar 30 2023, 12:48 PM

AWS_ENDPOINT -> LOCALSTACK_ENDPOINT

Harbormaster completed remote builds in B17845: Diff 24441.Mar 30 2023, 1:02 PM

varun added a reviewer: bartek.Mar 30 2023, 3:22 PM

bartek accepted this revision.Mar 31 2023, 4:17 AM

This revision is now accepted and ready to land.Mar 31 2023, 4:17 AM

Closed by commit rCOMM86ee2981a2e3: [identity] config for LocalStack (authored by varun). · Explain WhyMar 31 2023, 12:21 PM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM86ee2981a2e3: [identity] config for LocalStack.

Looks like i got distracted friday

scripts/source_development_defaults.sh
55 ↗	(On Diff #24441)	terraform seems to respect this environment variable, I don't think we should change it. Probably just define LOCALSTACK_ENDPOINT in addition to it.