Should not pass in mount option of awscredsuri

kubernetes-sigs · Aug 11, 2022 · bd0769e · bd0769e
1 parent d07d3e8
commit bd0769e
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/pkg/driver/node.go b/pkg/driver/node.go
@@ -78,7 +78,7 @@ func (d *Driver) NodePublishVolume(ctx context.Context, req *csi.NodePublishVolu
  switch strings.ToLower(k) {
  //Deprecated
  case "path":
- klog.Warning("Use of path under volumeAttributes is depracated. This field will be removed in future release")
+ klog.Warning("Use of path under volumeAttributes is deprecated. This field will be removed in future release")
  if !filepath.IsAbs(v) {
  return nil, status.Errorf(codes.InvalidArgument, "Volume context property %q must be an absolute path", k)
  }
@@ -165,6 +165,12 @@ func (d *Driver) NodePublishVolume(ctx context.Context, req *csi.NodePublishVolu
  }
  }
 
+ if f == "awscredsuri" {
+ klog.Warning("awscredsuri is not accepted as a efs-csi-driver mount option")
+ return nil, status.Errorf(codes.InvalidArgument,
+ "Should not pass in awscredsuri as a mount option")
+ }
+
  if !hasOption(mountOptions, f) {
  mountOptions = append(mountOptions, f)
  }