ArgumentParser superseding environment variables for options?

[rbonestell]

Hery @aaronshaf !

It seems that as a result of PR #447 that dynamodb-admin is no longer utilizing the environment variables to initialize the service.

My team uses your docker image for our local development stack, providing the following env vars:

environment:
  - DYNAMO_ENDPOINT=http://localstack:4566
  - AWS_REGION=us-west-2
  - AWS_ACCESS_KEY_ID=localstack
  - AWS_SECRET_ACCESS_KEY=localstack

Now, when dynamodb-admin v5.0.0 loads in docker, it prints the following:

2024-11-18 11:14:33   database endpoint:        http://localhost:8000
2024-11-18 11:14:33   region:           us-west-2
2024-11-18 11:14:33   accessKey:                localstack
2024-11-18 11:14:33 
2024-11-18 11:14:33   dynamodb-admin listening on http://:::8001 (alternatively http://0.0.0.0:8001)

This is likely due to the release v5.0.0 introducing ArgumentParser in bin/dynamodb-admin.ts and the default values may be overriding env vars?

[rchl]

ArgumentParser was there before too but I'll look at this regression.