Skip to content

feat(aws): Add IAM SSH Public Keys#5538

Merged
kodiakhq[bot] merged 3 commits intocloudquery:mainfrom
erezrokah:feat/iam_public_keys
Dec 12, 2022
Merged

feat(aws): Add IAM SSH Public Keys#5538
kodiakhq[bot] merged 3 commits intocloudquery:mainfrom
erezrokah:feat/iam_public_keys

Conversation

@erezrokah
Copy link
Copy Markdown
Member

Summary

Fixes #5519. In order to call ListSSHPublicKeys we need the user name, hence it being a relation of users.

This gets the key metadata, if we want the content too we need another API call to https://pkg.go.dev/github.com/aws/aws-sdk-go-v2/service/iam#Client.GetSSHPublicKey

@github-actions
Copy link
Copy Markdown

This PR has the following changes to source plugin(s) tables:

  • Table aws_iam_ssh_public_keys was added

Copy link
Copy Markdown
Contributor

@hermanschaaf hermanschaaf left a comment

Choose a reason for hiding this comment

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

LGTM, though I think having the public key in a column would be useful in many applications, so we may want to add that too.

@erezrokah
Copy link
Copy Markdown
Member Author

LGTM, though I think having the public key in a column would be useful in many applications, so we may want to add that too.

Thanks, I think we can always add it (won't be a breaking change). The use case from the issue is to enforce rotation, so I'm favoring for not doing another API call until someone requests it.

@erezrokah erezrokah added the automerge Automatically merge once required checks pass label Dec 12, 2022
@hermanschaaf
Copy link
Copy Markdown
Contributor

@erezrokah Yeah, not a blocking comment, sounds good 👍

@kodiakhq kodiakhq bot merged commit 5bd2b4d into cloudquery:main Dec 12, 2022
kodiakhq bot pushed a commit that referenced this pull request Dec 13, 2022
🤖 I have created a release *beep* *boop*
---


## [7.4.0](plugins-source-aws-v7.3.2...plugins-source-aws-v7.4.0) (2022-12-13)


### Features

* Add more cases where DateTime types in CSV responses of AWS IAM… ([#5526](#5526)) ([a43bf39](a43bf39))
* **aws:** Add IAM SSH Public Keys ([#5538](#5538)) ([5bd2b4d](5bd2b4d))


### Bug Fixes

* **aws-functions:** Return error in case of access denied ([#5537](#5537)) ([1fe79d3](1fe79d3))
* **aws:** Deterministic Multiplexer ([#5513](#5513)) ([eaf6e99](eaf6e99))
* **deps:** Update module github.com/aws/aws-sdk-go-v2/service/appstream to v1.18.2 ([#5566](#5566)) ([e7b7c24](e7b7c24))
* **deps:** Update module github.com/aws/aws-sdk-go-v2/service/athena to v1.20.2 ([#5523](#5523)) ([9385a69](9385a69))
* **deps:** Update module github.com/aws/aws-sdk-go-v2/service/autoscaling to v1.25.0 ([#5524](#5524)) ([5009a35](5009a35))
* **deps:** Update module github.com/aws/aws-sdk-go-v2/service/cloudwatchlogs to v1.17.2 ([#5535](#5535)) ([2ce06fc](2ce06fc))
* **deps:** Update module github.com/aws/aws-sdk-go-v2/service/ec2 to v1.76.0 ([#5569](#5569)) ([979a780](979a780))
* **deps:** Update module github.com/aws/aws-sdk-go-v2/service/mq to v1.13.17 ([#5567](#5567)) ([6075e71](6075e71))
* **deps:** Update module github.com/aws/aws-sdk-go-v2/service/rds to v1.34.0 ([#5570](#5570)) ([8fd69ba](8fd69ba))
* **deps:** Update module github.com/aws/aws-sdk-go-v2/service/wafv2 to v1.23.3 ([#5568](#5568)) ([1782f00](1782f00))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.12.0 ([#5539](#5539)) ([fb71293](fb71293))
* Update endpoints ([#5520](#5520)) ([88a88a7](88a88a7))
* Update endpoints ([#5565](#5565)) ([6b8c15d](6b8c15d))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
@erezrokah erezrokah deleted the feat/iam_public_keys branch December 14, 2022 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

IAM SSH public keys are not covered

3 participants