From b81863fb97fe6f48397d2ca7bfe23da3f01a2e3b Mon Sep 17 00:00:00 2001
From: Tanner Bruce <tanner.bruce@farmersedge.ca>
Date: Wed, 22 Nov 2017 07:36:05 -0600
Subject: [PATCH] Check error while walking environments
It is possible for the walk function to be called with err set
Signed-off-by: Tanner Bruce <bruce.tanner@gmail.com>
---
metadata/environment.go | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/metadata/environment.go b/metadata/environment.go
index 3b5d3899..3c3cea73 100644
--- a/metadata/environment.go
+++ b/metadata/environment.go
@@ -211,6 +211,10 @@ func (m *manager) GetEnvironments() ([]*Environment, error) {
log.Debug("Retrieving all environments")
err := afero.Walk(m.appFS, string(m.environmentsPath), func(path string, f os.FileInfo, err error) error {
+ if err != nil {
+ log.Debugf("Failed to walk the path at '%s'", path)
+ return err
+ }
isDir, err := afero.IsDir(m.appFS, path)
if err != nil {
log.Debugf("Failed to check whether the path at '%s' is a directory", path)
--
GitLab