Issue #32: HTTP2 support with latest Go

Drop the hard coded next protocols for the TLS config which defeats the auto configuration for both HTTP and HTTP2
fabiolb · Feb 7, 2016 · 35017de · 35017de
1 parent 62e3b29
commit 35017de
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,7 @@
 
 ### [v1.0.9] - Unreleased
 
+ * [Issue #32](https://github.com/eBay/fabio/issues/32): HTTP2 support with latest Go
  * [Issue #37](https://github.com/eBay/fabio/issues/37): Add support for consul ACL token to demo server
 
 ### [v1.0.8](https://github.com/eBay/fabio/releases/tag/v1.0.8) - 14 Jan 2015

diff --git a/listen.go b/listen.go
@@ -80,7 +80,6 @@ func newServer(l config.Listen, h http.Handler) (*http.Server, error) {
 		}
 
 		srv.TLSConfig = &tls.Config{
-			NextProtos:   []string{"http/1.1"},
 			Certificates: []tls.Certificate{cert},
 		}
 

diff --git a/listen_test.go b/listen_test.go
@@ -38,7 +38,6 @@ func TestNewServer(t *testing.T) {
 				Addr:    ":123",
 				Handler: h,
 				TLSConfig: &tls.Config{
-					NextProtos:   []string{"http/1.1"},
 					Certificates: []tls.Certificate{cert},
 				},
 			},
-Original file line number
+Diff line change
@@ Expand Up @@
     		}
     		srv.TLSConfig = &tls.Config{
-    			NextProtos:   []string{"http/1.1"},
     			Certificates: []tls.Certificate{cert},
     		}
@@ Expand Down @@