From 74696a19e0b5bc889f8872dc631fda02ac52b522 Mon Sep 17 00:00:00 2001
From: Richard LT <richard.le.terrier@gmail.com>
Date: Fri, 17 Dec 2021 10:51:49 +0100
Subject: [PATCH] fix(worker): executing plugin is failing for windows worker
 (#6042)

---
 sdk/fs.go | 23 ++++++++++++++++-------
 1 file changed, 16 insertions(+), 7 deletions(-)

diff --git a/sdk/fs.go b/sdk/fs.go
index a87530808c..e13aa57cf4 100644
--- a/sdk/fs.go
+++ b/sdk/fs.go
@@ -3,6 +3,7 @@ package sdk
 import (
 	"os"
 	"path/filepath"
+	"runtime"
 	"strings"
 
 	"github.com/pkg/errors"
@@ -17,12 +18,21 @@ var ErrExecutableNotFound = errors.New("executable file not found in $PATH")
 func findExecutable(fs afero.Fs, file string) error {
 	d, err := fs.Stat(file)
 	if err != nil {
-		return err
+		return errors.WithStack(err)
 	}
-	if m := d.Mode(); !m.IsDir() && m&0111 != 0 {
+
+	m := d.Mode()
+
+	isWindows := strings.EqualFold(runtime.GOOS, "windows")
+	isExecutable := m&0111 != 0
+
+	// Don't check exec permission for Windows.
+	// File mode will only return if the file is readonly or not.
+	// https://pkg.go.dev/os#Chmod
+	if !m.IsDir() && (isWindows || isExecutable) {
 		return nil
 	}
-	return os.ErrPermission
+	return errors.Wrapf(os.ErrPermission, "invalid permission %q", m)
 }
 
 // LookPath searches for an executable named file in the
@@ -35,11 +45,10 @@ func LookPath(fs afero.Fs, file string) (string, error) {
 	// but that would not match all the Unix shells.
 
 	if strings.Contains(file, "/") {
-		err := findExecutable(fs, file)
-		if err == nil {
-			return file, nil
+		if err := findExecutable(fs, file); err != nil {
+			return "", err
 		}
-		return "", ErrExecutableNotFound
+		return file, nil
 	}
 	path := os.Getenv("PATH")
 	for _, dir := range filepath.SplitList(path) {