Browse code

ContainerStats: return early on unsupported API versions

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

Sebastiaan van Stijn authored on 2019/07/11 04:47:50
Showing 1 changed files
... ...
@@ -21,6 +21,10 @@ func (daemon *Daemon) ContainerStats(ctx context.Context, prefixOrName string, c
21 21
 	// Engine API version (used for backwards compatibility)
22 22
 	apiVersion := config.Version
23 23
 
24
+	if runtime.GOOS == "windows" && versions.LessThan(apiVersion, "1.21") {
25
+		return errors.New("API versions pre v1.21 do not support stats on Windows")
26
+	}
27
+
24 28
 	container, err := daemon.GetContainer(prefixOrName)
25 29
 	if err != nil {
26 30
 		return err
... ...
@@ -70,9 +74,6 @@ func (daemon *Daemon) ContainerStats(ctx context.Context, prefixOrName string, c
70 70
 			var statsJSON interface{}
71 71
 			statsJSONPost120 := getStatJSON(v)
72 72
 			if versions.LessThan(apiVersion, "1.21") {
73
-				if runtime.GOOS == "windows" {
74
-					return errors.New("API versions pre v1.21 do not support stats on Windows")
75
-				}
76 73
 				var (
77 74
 					rxBytes   uint64
78 75
 					rxPackets uint64