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
SQS VPC Endpoint in c7n_mailer issue 5722 #5770
Conversation
thanks for the pull request, github actions seems to be having issues atm. |
tools/c7n_mailer/README.md
Outdated
@@ -257,6 +257,7 @@ and here is a description of the options: | |||
|:---------:|:----------------|:-----------------|:------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------| | |||
| ✅ | `queue_url` | string | the queue to listen to for messages | | |||
| | `from_address` | string | default from address | | |||
| | `endpoint_url` | string | The complete URL to use for the constructed client. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor doc string here, probably should read - SQS API URL (for use with VPC Endpoints)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
one minor on doc string else looks good to me, thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks
issue 5722 attempting to add in the endpoint_url into the configuration and have it part of the session's client call. If it is not defined, it will default to null and in the call will be = to None which is what https://boto3.amazonaws.com/v1/documentation/api/latest/reference/core/session.html
seems to be expecting. Added in documentation change.