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

Add support for AWS dynamodb #164

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

samuelabreu
Copy link

Using url params to set profile and endpoint url if needed.

Using url params to set profile and endpoint url if needed.
README.markdown Outdated
Comment on lines 51 to 57
DynamoDB uses profiles [Configuration and credential file settings][] as
authentication

* :DB dynamodb:// # use default profile
* :DB dynamodb://profile_name
* :DB dynamodb://profile_name@localhost:8888 # To connect on a local dynamodb instance

Copy link
Owner

Choose a reason for hiding this comment

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

This should be explained in the help file, not the README.

@SuperCuber SuperCuber mentioned this pull request May 16, 2024
@augustdolan
Copy link

@samuelabreu do you have success with all the possible cli commands? If I run a query command in my terminal, I receive a successful response. However, when I run the equivalent command with dadbod I get:

usage: aws [options] <command> <subcommand> [<subcommand> ...] [parameters]
To see help text, you can run:

  aws help
  aws <command> help
  aws <command> <subcommand> help

Unknown options: :pk", {"S":, "myPrimaryKey"}}', =

aws cli command: aws dynamodb --profile myProfile query --key-condition-expression "PK = :pk" --expression-attribute-values '{":pk": {"S": "myQueryKey"}}' --table-name BaseStackdevelopment-PsSingleTable

DB command: query --key-condition-expression "PK = :pk" --expression-attribute-values '{":pk": {"S": "myQueryKey"}}' --table-name BaseStackdevelopment-PsSingleTable

For context, dadbod is able to successfully run a list-tables command

@samuelabreu
Copy link
Author

@augustdolan , i only used for basic commands, but i just push a commit and test it with your query.

@augustdolan
Copy link

@augustdolan , i only used for basic commands, but i just push a commit and test it with your query.

Success! I am now able to run full commands that use quotes! Thanks so much for pushing up the new commit!

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

Successfully merging this pull request may close these issues.

3 participants