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

Default to sig v4 #82

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

c960657
Copy link

@c960657 c960657 commented Jan 13, 2021

Signature Version 2 will be retired on March 27th, 2021. Let's default to v4, so users wont have to explicitly force this version.

@@ -42,7 +42,9 @@ module SES
DEFAULT_HOST = 'email.us-east-1.amazonaws.com'

DEFAULT_MESSAGE_ID_DOMAIN = 'email.amazonses.com'


DEFAULT_SIGNATURE_VERSION = 4

Choose a reason for hiding this comment

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

Dev from AWS SES here. Thanks for this initiative. Sigv4 is not just going to be the default, it's going to be the only signing method supported in the near future, so I don't think backward compatibility applies here, only Sigv4 can remain

@girisha-g
Copy link

@sleg-aws by when can you merge this PR?

@sleg-aws
Copy link

sleg-aws commented Apr 6, 2021

@girisha-g I don't own this repo. This is not a AWS-owned repository. You may contact people with write-access to this repo.

@girisha-g
Copy link

@drewblas can you merge and release this PR?
I am using this gem in my RoR application and I am getting warnings

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