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

Use TLS by default #171

Merged
merged 4 commits into from
Mar 23, 2021
Merged

Use TLS by default #171

merged 4 commits into from
Mar 23, 2021

Conversation

elverkilde
Copy link
Contributor

Description

Any major version now switches to use TLS by default. It's 2021 after all. This will be the final PR before preparing the major version bump, hence the longer changelog below.

The full diff between the last tag is here: https://github.com/pusher/pusher-http-ruby/compare/v1.4.3...ssl_is_default?expand=1

CHANGELOG

  • [CHANGED] Use TLS by default.
  • [REMOVED] Support for Ruby 2.4 and 2.5.
  • [FIXED] Handle empty or nil cluster configuration.
  • [REMOVED] Legacy Push Notification integration.
  • [ADDED] Stalebot and Github actions.

@elverkilde elverkilde force-pushed the ssl_is_default branch 2 times, most recently from cb27b29 to d549f43 Compare March 23, 2021 10:07
@elverkilde elverkilde added the release-major Major version bump label Mar 23, 2021
Copy link
Contributor

@GeorgeRDBuckley GeorgeRDBuckley left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@elverkilde elverkilde added release-major Major version bump and removed release-major Major version bump labels Mar 23, 2021
@elverkilde elverkilde added release-major Major version bump and removed release-major Major version bump labels Mar 23, 2021
@elverkilde elverkilde added release-major Major version bump and removed release-major Major version bump labels Mar 23, 2021
@elverkilde elverkilde merged commit 8b2f5b1 into master Mar 23, 2021
@elverkilde elverkilde deleted the ssl_is_default branch March 23, 2021 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-major Major version bump
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants