Skip to content

Commit 5e4bd60

Browse files
authored
Enable ALPN over TLS for H2 by default (#3213)
1 parent 18378ca commit 5e4bd60

File tree

3 files changed

+7
-5
lines changed

3 files changed

+7
-5
lines changed

gems/aws-sdk-core/CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
Unreleased Changes
22
------------------
33

4+
* Issue - Enable ALPN over TLS for H2 by default.
5+
46
3.220.1 (2025-03-06)
57
------------------
68

gems/aws-sdk-core/lib/seahorse/client/plugins/h2.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -53,10 +53,10 @@ class H2 < Plugin
5353
When `true`, HTTP2 debug output will be sent to the `:logger`.
5454
DOCS
5555

56-
option(:enable_alpn, default: false, doc_type: 'Boolean', docstring: <<-DOCS)
57-
Set to `true` to enable ALPN in HTTP2 over TLS. Requires Openssl version >= 1.0.2.
58-
Defaults to false. Note: not all service HTTP2 operations supports ALPN on server
59-
side, please refer to service documentation.
56+
option(:enable_alpn, default: true, doc_type: 'Boolean', docstring: <<-DOCS)
57+
Set to `false` to disable ALPN in HTTP2 over TLS. ALPN requires Openssl version >= 1.0.2.
58+
Note: RFC7540 requires HTTP2 to use ALPN over TLS but some
59+
services may not fully support ALPN and require setting this to `false`.
6060
DOCS
6161

6262
option(:logger)

gems/aws-sdk-core/spec/seahorse/client/plugins/h2_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ module Plugins
7272
end
7373

7474
it 'adds a :enable_alpn option with default' do
75-
expect(config.enable_alpn).to eq(false)
75+
expect(config.enable_alpn).to eq(true)
7676
end
7777

7878
it 'defaults the :raise_response_errors to true' do

0 commit comments

Comments
 (0)