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

[AWS] Support custom endpoints for Kinesis and Firehose. #7317

Merged
merged 1 commit into from
Sep 28, 2021
Merged

[AWS] Support custom endpoints for Kinesis and Firehose. #7317

merged 1 commit into from
Sep 28, 2021

Conversation

OlegOAndreev
Copy link
Contributor

Added --aws_firehose_endpoint and --aws_kinesis_endpoint flags to support non-AWS Kinesis and Firehose implementations. Setting the flag disables AWS region validation.

@OlegOAndreev OlegOAndreev requested review from a team as code owners September 21, 2021 21:49
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Sep 21, 2021

CLA Signed

The committers are authorized under a signed CLA.

@mike-myers-tob mike-myers-tob added AWS Related to Amazon Web Services tables logging ready for review Pull requests that are ready to be reviewed by a maintainer labels Sep 23, 2021
@mike-myers-tob
Copy link
Member

Thanks @OlegOAndreev ! Good to see a new contributor.

There's a documentation page about the AWS logging plugin here https://osquery.readthedocs.io/en/latest/deployment/aws-logging/

It's generated from this file: https://github.com/osquery/osquery/blob/master/docs/wiki/deployment/aws-logging.md

Do you think you could add some brief documentation of the new flags, to your PR?

@OlegOAndreev
Copy link
Contributor Author

OlegOAndreev commented Sep 24, 2021

Thanks for the quick reply! I've mentioned the new flags in the docs and fixed the build.

Copy link
Member

@zwass zwass left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@zwass zwass merged commit e628ef6 into osquery:master Sep 28, 2021
@mike-myers-tob mike-myers-tob removed the ready for review Pull requests that are ready to be reviewed by a maintainer label Oct 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AWS Related to Amazon Web Services tables logging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants