Update bucket name validation (merge commit)

Merge branch 'feature/update-bucket-creation' into 'main'
* Switch to stricter bucket name validation

See merge request https://gitlab.ci.csc.fi/sds-dev/sd-connect/swift-browser-ui/-/merge_requests/450

Reviewed-by: Sampsa Penna <sapenna@csc.fi>
Approved-by: Timo Nurminen <tnurmine@csc.fi>
Merged by Monika Radaviciute <mradavic@csc.fi>

Author: mradavi

.github/config/.finnishwords.txt

@@ -1,10 +1,13 @@
+aakkosia
 aika
 aikana
 aina
 ajaksi
+aksenttimerkkejä
 aktiiviset
 alapuolisesta
 alaviiva
+alkaa
 alkanut
 alla
 aloitettiin
@@ -109,6 +112,7 @@ ilmoittaa
 ilmoittaminen
 ilmoituksen
 informaatio
+isot
 ja
 jaa
 jaan
@@ -173,6 +177,7 @@ jäsenet
 jäsenille
 jäsenten
 kaikille
+kaikissa
 kaikki
 kaikkien
 kansio
@@ -199,6 +204,7 @@ kelvollisia
 kenttää
 kentällä
 kerran
+kesken
 keskeyttää
 keskus
 kestää
@@ -207,6 +213,8 @@ kielletty
 kiellettyä
 kielletyn
 kiinni
+kirjaimella
+kirjaimet
 kirjaudu
 kirjautua
 kirjautuaksesi
@@ -308,10 +316,12 @@ laskutuksesta
 laskutusyksiköiden
 laskutusysiköiden
 lataa
+lataamisen
 latauksen
 lataukset
 lataus
 latauslinkki
+latinalaisia
 leikepöydälle
 liiku
 liitä
@@ -332,9 +342,11 @@ luettelo
 luku
 lukuoikeus
 luo
+luodaan
 luomisen
 luonnin
 luonti
+luottamuksellisia
 luotu
 lupa
 lähetettiin
@@ -404,14 +416,18 @@ navigoitaessa
 ne
 niiden
 niin
+niissä
 nimellä
 nimen
 nimet
 nimetä
 nimeä
 nimi
+nimiä
 numeroinen
 numeroisen
+numeroita
+numerolla
 nykyinen
 nykyisen
 nykyisiä
@@ -520,12 +536,15 @@ pelkästä
 peruuta
 peruutettu
 pian
+pienellä
 pienemmissä
 pienennä
 pienillä
+pieniä
 piilota
 piste
 pitkiä
+pitkä
 poiketa
 pois
 poissa
@@ -593,6 +612,7 @@ päivätty
 päänäkymään
 pääsy
 pääsyä
+päätasolle
 päättyä
 rajan
 rajattu
@@ -761,6 +781,7 @@ tiedot
 tieteen
 tieto
 tietoa
+tietoja
 tietosivulta
 tietotekniikan
 tietotekniikkakeskus
@@ -837,6 +858,7 @@ ulkopuolinen
 ulos
 umpeutui
 uniikin
+uniikki
 useiden
 useille
 useisiin
@@ -912,6 +934,7 @@ vuoksi
 vähintään
 väliaikaiseen
 väliaikaisten
+väliviivaa
 vältä
 yhdelle
 yhden
@@ -939,6 +962,7 @@ ylläpitäjälle
 ymmärrettävyys
 yritit
 yritä
+ämpäreiden
 ämpäreille
 ämpäreitä
 ämpäri
@@ -949,3 +973,4 @@ yritä
 ämpäristä
 ämpärit
 ämpäriä
+åäöé

swift_browser_ui_frontend/src/common/api.js

@@ -375,6 +375,22 @@ export async function swiftCreateContainer(
   }
 }
 
+export async function swiftCheckContainerExists(
+  project,
+  container,
+) {
+  const fetchURL = new URL(
+    "/api/".concat(
+      encodeURI(project), "/",
+      encodeURI(container),
+    ),
+    document.location.origin,
+  );
+  const ret = await GET(fetchURL);
+  if (ret.status === 200 || ret.status === 403) return true;
+  if (ret.status === 404) return false;
+}
+
 export async function swiftDeleteContainer(
   project,
   container,

swift_browser_ui_frontend/src/common/globalFunctions.js

@@ -1,6 +1,7 @@
 import store from "@/common/store";
 import { taginputConfirmKeys } from "@/common/conv";
 import { getDB } from "@/common/db";
+import { swiftCheckContainerExists } from "@/common/api";
 
 export function toggleCreateBucketModal() {
   store.commit("toggleCreateBucketModal", true);
@@ -135,27 +136,29 @@ export function getPaginationOptions(t) {
   return paginationOptions;
 }
 
-export function validateBucketName(str, t, containers) {
-  //minimum length 3 chars
-  let error = "";
-  //forbid !"#$%&'()*+,/:;<=>?@[\]^`{|}~  allow .-_
-  const re = new RegExp("[!-,/:-@\\[-\\^`\\{-~]");
-  if (str.length <= 2) {
-    error = t("message.error.tooShort");
-  }
-  else if (str.match(re)) {
-    error= t("message.error.forbiddenChars");
-  }
-  else if (str.endsWith("_segments")) {
-    error= t("message.error.segments");
-  }
-  else {
-    if (containers) {
-      const found = containers.find(cont => cont.name === str);
-      if (found) error = t("message.error.inUse");
-    }
+export async function validateBucketName(input) {
+  let result = {
+    lowerCaseOrNum: undefined,
+    inputLength: undefined,
+    alphaNumHyphen: undefined,
+    ownable: undefined,
+  };
+  if (!input) return result;
+
+  result.lowerCaseOrNum = !!(
+    input[0].match(/[\p{L}0-9]/u) && input[0] === input[0].toLowerCase()
+  );
+  result.inputLength = input.length >= 3 && input.length <= 63;
+  result.alphaNumHyphen = !!input.match(/^[a-z0-9-]+$/g);
+  if (result.lowerCaseOrNum && result.inputLength && result.alphaNumHyphen) {
+    const containerExists = await swiftCheckContainerExists(
+      store.state.active.id, input);
+    // In undefined case allow user to attempt bucket creation
+    result.ownable = !containerExists;
+  } else {
+    result.ownable = false;
   }
-  return error;
+  return result;
 }
 
 export function getCurrentISOtime(time) {

swift_browser_ui_frontend/src/common/lang.js

@@ -250,20 +250,30 @@ let default_translations = {
       tagName: "Tags (optional)",
       tagPlaceholder: "# Add a tag and press enter",
       container_ops: {
-        addContainer: "Create new bucket",
-        norename:
-          "Please note that bucket names cannot be modified " +
-          "after creating a bucket.",
+        addContainer: "Create a new bucket",
         createdBucket:
           "Created bucket will be shared with all project members in ",
         viewProjectMembers: "View project members",
         deleteNote: "Bucket must be empty before " + "it can be deleted.",
         deleteSuccess: "Bucket was deleted.",
         bucketName: "Bucket name",
       },
+      nameValidation: {
+        lowerCaseOrNum:
+          "Bucket name should start with a lowercase letter or a number.",
+        inputLength:
+          "Bucket name should be between 3 and 63 characters long.",
+        alphaNumHyphen1:
+          "Use only lowercase Latin letters (a-z), numbers (0-9), and " +
+          "hyphens (-).",
+        alphaNumHyphen2: "Uppercase letters, underscore (_) and accent " +
+        "letters with diacritics or special marks (åäöé) are not allowed.",
+        ownable: "Bucket names must be unique across all existing buckets " +
+        "in all projects.",
+      },
       folders: {
         deleteNote:
-          "Folders are deleted by deleting all " + "items in them.",
+          "Folders are deleted by deleting all items in them.",
         deleteOneSuccess: "Folder was deleted.",
         deleteManySuccess: "Folders were deleted.",
       },
@@ -320,8 +330,15 @@ let default_translations = {
       encrypt: {
         uploadFiles: "Upload files",
         uploadDestination: "Destination bucket: ",
-        upload_step1: "Create a new bucket",
-        upload_step2: "Files to be uploaded",
+        uploadStep1: {
+          title: "Create a new bucket",
+          createAtRoot: "Bucket is created at the root level of your project.",
+          nonModifiable:
+            "Bucket names cannot be changed after " +
+            "creation or upload. Remember, all bucket names are public; " +
+            "please do not include any confidential information.",
+        },
+        uploadStep2: "Files to be uploaded",
         multipleReceivers: "Additional encryption keys (public keys only)",
         pubkey: "Paste public key",
         pubkeyLabel: "Public keys (SHA-256)",
@@ -802,9 +819,6 @@ let default_translations = {
       tagPlaceholder: "# Lisää asiasana ja paina rivinvaihtoa",
       container_ops: {
         addContainer: "Luo uusi ämpäri",
-        norename:
-          "Ämpäriä ei voi nimetä uudelleen, " +
-          "mutta sen voi kopioida uudella nimellä.",
         createdBucket: "Luotu ämpäri jaetaan kaikille jäsenille projektissa ",
         viewProjectMembers: "Näytä projektin jäsenet",
         deleteNote:
@@ -813,6 +827,18 @@ let default_translations = {
         deleteSuccess: "Ämpäri poistettu",
         bucketName: "Ämpärin nimi",
       },
+      nameValidation: {
+        lowerCaseOrNum:
+          "Ämpärin nimi alkaa pienellä kirjaimella tai numerolla.",
+        inputLength: "Ämpärin nimi on 3-63 merkkiä pitkä.",
+        alphaNumHyphen1: "Käytä vain latinalaisia pieniä aakkosia (a-z), " +
+        "numeroita (0-9) ja väliviivaa (-).",
+        alphaNumHyphen2: "Isot kirjaimet, alaviiva (_) ja kirjaimet, " +
+        "joissa on aksenttimerkkejä tai erikoismerkkejä (åäöé) eivät " +
+        "ole sallittuja.",
+        ownable: "Ämpärin nimen tulee olla uniikki kaikkien ämpäreiden " +
+        "kesken kaikissa projekteissa.",
+      },
       folders: {
         deleteNote:
           "Kansion poistaminen edellyttää sen kaikkien " +
@@ -874,8 +900,14 @@ let default_translations = {
       encrypt: {
         uploadFiles: "Lataa tiedostoja",
         uploadDestination: "Kohdeämpäri: ",
-        upload_step1: "Luo uusi ämpäri",
-        upload_step2: "Ladattavat tiedostot",
+        uploadStep1: {
+          title: "Luo uusi ämpäri",
+          createAtRoot: "Ämpäri luodaan projektin päätasolle.",
+          nonModifiable: "Ämpärin nimiä ei voi " +
+          "muokata luomisen tai lataamisen jälkeen. Ämpäreiden nimet ovat " +
+          "julkisia ja niissä ei tulisi käyttää luottamuksellisia tietoja.",
+        },
+        uploadStep2: "Ladattavat tiedostot",
         multipleReceivers: "Lisää salausavaimia (vain julkiset avaimet)",
         addkey: "Lisää avain",
         pubkey: "Liitä julkinen avain",