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

describe_db_instances db_instance_identifier option does not function properly #8

Closed
fmcgeough opened this issue Aug 8, 2018 · 0 comments

Comments

@fmcgeough
Copy link
Collaborator

Option to describe_db_instances doesn't return expected values.

The following code will work:

  def describe_rds_instance(rds_db_identifier, keys) do 
    options = [{:DBInstanceIdentifier,  rds_db_identifier}]
    ExAws.RDS.describe_db_instances(options) |> ExAws.request(keys)
  end

but describe_db_instances_opts type has:

{:db_instance_identifier, binary}

If this is passed in as a parameter its ignored and every RDS instance for your requested account/region will be returned. The code currently ends up sending "DbInstanceIdentifier" and since the API is case-sensitive it doesn't match on this key.

fmcgeough added a commit to fmcgeough/ex_aws_rds that referenced this issue Oct 22, 2018
Fixes Issue ex-aws#8. The option is renamed so its name matches what
AWS expects.
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

No branches or pull requests

1 participant