Handle eos EPERM as permission denied (#1183)

Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
cs3org · Sep 21, 2020 · d635375 · d635375
1 parent 84f2c0e
commit d635375
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/changelog/unreleased/handle-eos-eperm.md b/changelog/unreleased/handle-eos-eperm.md
@@ -0,0 +1,5 @@
+Bugfix: Handle eos EPERM as permission denied
+
+We now treat EPERM errors, which occur, eg. when acl checks fail and return a permission denied error.
+
+https://github.com/cs3org/reva/pull/1183
diff --git a/pkg/eosclient/eosclient.go b/pkg/eosclient/eosclient.go
@@ -247,7 +247,7 @@ func (c *Client) executeEOS(ctx context.Context, cmd *exec.Cmd) (string, string,
 				err = nil
 			case 2:
 				err = errtypes.NotFound(errBuf.String())
-			case 22:
+			case 1, 22:
 				// eos reports back error code 22 when the user is not allowed to enter the instance
 				err = errtypes.PermissionDenied(errBuf.String())
 			}