Переглянути джерело

Fix panic when invoking inspect with empty container id (ACI SDK returns status 200 but nil container group properties)

Signed-off-by: Guillaume Tardif <[email protected]>
Guillaume Tardif 5 роки тому
батько
коміт
d8cd4038c2
1 змінених файлів з 4 додано та 1 видалено
  1. 4 1
      aci/containers.go

+ 4 - 1
aci/containers.go

@@ -235,12 +235,15 @@ func (cs *aciContainerService) Delete(ctx context.Context, containerID string, r
 
 func (cs *aciContainerService) Inspect(ctx context.Context, containerID string) (containers.Container, error) {
 	groupName, containerName := getGroupAndContainerName(containerID)
+	if containerID == "" {
+		return containers.Container{}, errors.New("cannot inspect empty container ID")
+	}
 
 	cg, err := getACIContainerGroup(ctx, cs.ctx, groupName)
 	if err != nil {
 		return containers.Container{}, err
 	}
-	if cg.StatusCode == http.StatusNoContent {
+	if cg.IsHTTPStatus(http.StatusNoContent) || cg.ContainerGroupProperties == nil || cg.ContainerGroupProperties.Containers == nil {
 		return containers.Container{}, errdefs.ErrNotFound
 	}