Using web console to set server location adds space before value making it invalid

[jsabater]

When using the Settings: Region: Server Location option in the web console to set a value (e.g. FSN1-DC3 in my case), a space character at the beginning of the value is somehow added, which renders it invalid. So, when MinIO starts it complains about it:

Error: Invalid site configuration: region ' FSN1-DC3' is invalid, expected simple characters such as [us-east-1, myregion...] (*fmt .wrapError)

Note the extra space character at the begining of the value.

Expected Behavior

Not adding an extra space character at the beginning. Moreover, ideally the UI should check the value before trying to save it (against the same rules MinIO checks it when starting up).

Current Behavior

An error from minio is logged into /var/log/syslog, but the server starts and works correctly.

Your Environment

It's a very recent version, but not the latest:

# minio --version
minio version RELEASE.2023-02-27T18-10-45Z (commit-id=0ff931dc765d5d8fdddcdf7687fec1b0b194fca9)
Runtime: go1.19.6 linux/amd64
License: GNU AGPLv3 <https://www.gnu.org/licenses/agpl-3.0.html>
Copyright: 2015-2023 MinIO, Inc.

It's running inside an LXC on a Proxmox 7.4 server on Debian 11 Bullseye:

# uname -a
Linux minio1 5.15.104-1-pve minio/minio#1 SMP PVE 5.15.104-1 (2023-03-29T15:51Z) x86_64 GNU/Linux

[harshavardhana]

Thanks for the issue @jsabater we will take a look.

[cesnietor]

This is the screen, we'll try to reproduce.

[cniackz]

Hello team,

I was able to reproduce the issue in version: RELEASE.2023-06-29T05-12-28Z
And I will attempt to fix this problem by removing the whitespace prior saving:

API: SYSTEM()
Time: 14:05:47 UTC 06/30/2023
DeploymentID: ad79aacc-3ffb-4726-9ff3-6cc7ca58b27a
Error: Invalid site configuration: region ' us-west-rack-2' is invalid, expected simple characters such as [us-east-1, myregion...] (*fmt.wrapError)
       7: internal/logger/logger.go:258:logger.LogIf()
       6: cmd/config-current.go:493:cmd.lookupConfigs()
       5: cmd/config.go:218:cmd.initConfig()
       4: cmd/config.go:189:cmd.(*ConfigSys).Init()
       3: cmd/server-main.go:452:cmd.initConfigSubsystem()
       2: cmd/server-main.go:423:cmd.initServerConfig()
       1: cmd/server-main.go:662:cmd.serverMain()

[cniackz]

The solution could be:

$ git diff src/screens/Console/Configurations/utils.tsx
diff --git a/portal-ui/src/screens/Console/Configurations/utils.tsx b/portal-ui/src/screens/Console/Configurations/utils.tsx
index c386eb3b9..f8ae1076d 100644
--- a/portal-ui/src/screens/Console/Configurations/utils.tsx
+++ b/portal-ui/src/screens/Console/Configurations/utils.tsx
@@ -401,8 +401,12 @@ export const fieldsConfigurations: any = {
 };
 
 export const removeEmptyFields = (formFields: IElementValue[]) => {
+  // Remove white spaces from strings
+  for (let i = 0; i < formFields.length; i++) {
+    // @ts-ignore
+    formFields.at(i).value = formFields.at(i).value.trim();
+  }
   const nonEmptyFields = formFields.filter((field) => field.value !== "");
-
   return nonEmptyFields;
 };