Dockerfile

@@ -1,4 +1,4 @@
-FROM golang:1.21.0 as build
+FROM golang:1.20.6 as build
 
 LABEL author="Victor Lacasse-Beaudoin <vlbeaudoin@agecem.org>"
 
@@ -24,7 +24,9 @@ RUN CGO_ENABLED=0 go build -a -installsuffix cgo -o agecem-org .
 
 # Alpine
 
-FROM alpine:3.18.3
+FROM alpine:3.17.2
+
+RUN apk update && apk upgrade --no-cache
 
 WORKDIR /app
 

cmd/server.go

@@ -96,12 +96,7 @@ func init() {
 	viper.BindPFlag("server.documents.use_ssl", serverCmd.Flags().Lookup("server-documents-use-ssl"))
 
 	// server.documents.buckets    -    --server-documents-buckets
-	serverCmd.Flags().StringToString("server-documents-buckets", map[string]string{
+	serverCmd.Flags().StringSlice("server-documents-buckets", []string{"proces-verbaux", "politiques-et-reglements"}, "Buckets that are allowed to be accessed by the API (config: server.documents.buckets)")
-		"proces-verbaux": "Procès-verbaux",
-		"politiques":     "Politiques",
-		"reglements":     "Règlements",
-		"formulaires":    "Formulaires",
-	}, "Buckets that are allowed to be accessed by the API (config: server.documents.buckets)")
 	viper.BindPFlag("server.documents.buckets", serverCmd.Flags().Lookup("server-documents-buckets"))
 
 	// server.api.auth    -    --server-api-auth
@@ -298,7 +293,7 @@ func handleDocumentation(c echo.Context) error {
 		return c.Render(http.StatusInternalServerError, "documentation-html", nil)
 	}
 
-	var buckets map[string]string
+	var buckets []string
 
 	err = json.Unmarshal(result, &buckets)
 	if err != nil {
@@ -307,13 +302,12 @@ func handleDocumentation(c echo.Context) error {
 
 	type Bucket struct {
 		Name      string
-		DisplayName string
 		Documents []string
 	}
 
 	var data []Bucket
 
-	for bucket, displayName := range buckets {
+	for _, bucket := range buckets {
 		content, err := client.Call(http.MethodGet, fmt.Sprintf("/v1/bucket/%s", bucket))
 		if err != nil {
 			return c.Render(http.StatusInternalServerError, "documentation-html", nil)
@@ -346,7 +340,6 @@ func handleDocumentation(c echo.Context) error {
 
 		data = append(data, Bucket{
 			Name:      bucket,
-			DisplayName: displayName,
 			Documents: documents,
 		})
 	}
@@ -388,56 +381,10 @@ func handleAdmin(c echo.Context) error {
 }
 
 func handleAdminDocumentsUpload(c echo.Context) error {
-	client, err := api.NewApiClientFromViper()
+	return c.Render(http.StatusOK, "admin-upload-html", nil)
-	if err != nil {
-		return c.Render(http.StatusInternalServerError, "documentation-html", nil)
-	}
-
-	result, err := client.Call(http.MethodGet, "/v1/bucket")
-	if err != nil {
-		return c.Render(http.StatusInternalServerError, "documentation-html", nil)
-	}
-
-	var buckets map[string]string
-
-	err = json.Unmarshal(result, &buckets)
-	if err != nil {
-		return c.Render(http.StatusInternalServerError, "documentation-html", nil)
-	}
-
-	type Bucket struct {
-		Name        string
-		DisplayName string
-		Documents   []string
-	}
-
-	var data struct {
-		Buckets []Bucket
-		Message string
-	}
-
-	for bucketName, displayName := range buckets {
-		data.Buckets = append(data.Buckets, Bucket{
-			Name:        bucketName,
-			DisplayName: displayName,
-		})
-	}
-
-	return c.Render(http.StatusOK, "admin-upload-html", data)
 }
 
 func handleAdminDocumentsUploadPOST(c echo.Context) error {
-	type Bucket struct {
-		Name        string
-		DisplayName string
-		Documents   []string
-	}
-
-	var data struct {
-		Buckets []Bucket
-		Message string
-	}
-
 	client, err := api.New(cfg.Server.Api.Protocol, cfg.Server.Api.Host, cfg.Server.Port, api.APIOptions{
 		KeyAuth:   cfg.Server.Api.Auth,
 		Key:       cfg.Server.Api.Key,
@@ -446,52 +393,29 @@ func handleAdminDocumentsUploadPOST(c echo.Context) error {
 		Password:  cfg.Server.Admin.Password,
 	})
 	if err != nil {
-		data.Message = fmt.Sprintf("handleAdminDocumentsUploadPOST#api.New: %s", err)
+		return c.Render(http.StatusInternalServerError, "admin-upload-html", struct{ Message string }{Message: fmt.Sprintf("handleAdminDocumentsUploadPOST#api.New: %s", err)})
-		return c.Render(http.StatusInternalServerError, "admin-upload-html", data)
-	}
-
-	result, err := client.Call(http.MethodGet, "/v1/bucket")
-	if err != nil {
-		data.Message = "Error during GET /v1/bucket"
-		return c.Render(http.StatusInternalServerError, "documentation-html", data)
-	}
-
-	var buckets map[string]string
-
-	err = json.Unmarshal(result, &buckets)
-	if err != nil {
-		return c.Render(http.StatusInternalServerError, "documentation-html", nil)
-	}
-
-	for bucketName, displayName := range buckets {
-		data.Buckets = append(data.Buckets, Bucket{
-			Name:        bucketName,
-			DisplayName: displayName,
-		})
 	}
 
 	bucket := c.FormValue("bucket")
 
 	document, err := c.FormFile("document")
 	if err != nil {
-		data.Message = fmt.Sprintf("handleAdminDocumentsUploadPOST#c.FormFile: %s", err)
+		return c.Render(http.StatusBadRequest, "admin-upload-html", struct{ Message string }{Message: fmt.Sprintf("handleAdminDocumentsUploadPOST#c.FormFile: %s", err)})
-		return c.Render(http.StatusBadRequest, "admin-upload-html", data)
 	}
 
 	response, err := client.UploadDocument(bucket, document)
 	if err != nil {
-		data.Message = fmt.Sprintf("handleAdminDocumentsUploadPOST#client.UploadDocument: %s", err)
+		return c.Render(http.StatusInternalServerError, "admin-upload-html", struct{ Message string }{Message: fmt.Sprintf("handleAdminDocumentsUploadPOST#client.UploadDocument: %s", err)})
-		return c.Render(http.StatusInternalServerError, "admin-upload-html", data)
 	}
 
 	// Format response
-	var info, status string
+	var message, info, status string
 
 	info = fmt.Sprintf("[%.0f] /public/documentation/%s/%s", response.Info.Size, response.Info.Bucket, response.Info.Object)
 
 	status = response.Message
 
-	data.Message = fmt.Sprintf("%s - %s", status, info)
+	message = fmt.Sprintf("%s - %s", status, info)
 
-	return c.Render(http.StatusOK, "admin-upload-html", data)
+	return c.Render(http.StatusOK, "admin-upload-html", struct{ Message string }{Message: message})
 }

config/config.go

@@ -29,7 +29,7 @@ type Config struct {
 		} `mapstructure:"api"`
 		Documents struct {
 			AccessKeyId     string   `mapstructure:"access_key_id"`
-			Buckets         map[string]string `mapstructure:"buckets"`
+			Buckets         []string `mapstructure:"buckets"`
 			Endpoint        string   `mapstructure:"endpoint"`
 			SecretAccessKey string   `mapstructure:"secret_access_key"`
 			UseSSL          bool     `mapstructure:"use_ssl"`

examples/config/agecem-org.yaml

@@ -35,10 +35,8 @@ server:
     #
     # Also used to specify which buckets are to be created on receiving a POST request on /v1/seed
     buckets:
-      'proces-verbaux': 'Procès-verbaux'
+      - 'proces-verbaux'
-      'politiques': 'Politiques'
+      - 'politiques-et-reglements'
-      'reglements': 'Règlements'
-      'formulaires': 'Formulaires'
 
   api:
     # Enable or disable key auth on /v1 routes

media/media.go

@@ -58,7 +58,7 @@ func (m *MediaClient) Seed() ([]string, error) {
 
 	var new_buckets []string
 
-	for bucket := range cfg.Server.Documents.Buckets {
+	for _, bucket := range cfg.Server.Documents.Buckets {
 		exists, err := m.MinioClient.BucketExists(context.Background(), bucket)
 		if err != nil {
 			return new_buckets, err

serverhandlers/serverhandlers.go

@@ -72,16 +72,16 @@ func HandleV1BucketList(c echo.Context) error {
 		})
 	}
 
-	var buckets = make(map[string]string)
+	var buckets []string
 
-	for bucket_name, bucket_display_name := range cfg.Server.Documents.Buckets {
+	for _, bucket_name := range cfg.Server.Documents.Buckets {
 		exists, err := mediaClient.MinioClient.BucketExists(context.Background(), bucket_name)
 		if err != nil {
 			return c.JSON(http.StatusInternalServerError, "Error during minio#BucketExists")
 		}
 
 		if exists {
-			buckets[bucket_name] = bucket_display_name
+			buckets = append(buckets, bucket_name)
 		}
 	}
 
@@ -99,7 +99,7 @@ func HandleV1BucketRead(c echo.Context) error {
 	bucket := c.Param("bucket")
 
 	allowed := false
-	for bucket_allowed := range cfg.Server.Documents.Buckets {
+	for _, bucket_allowed := range cfg.Server.Documents.Buckets {
 		if bucket == bucket_allowed {
 			allowed = true
 		}
@@ -171,7 +171,7 @@ func HandleV1DocumentCreate(c echo.Context) error {
 	}
 
 	allowed := false
-	for bucket_allowed := range cfg.Server.Documents.Buckets {
+	for _, bucket_allowed := range cfg.Server.Documents.Buckets {
 		if bucket == bucket_allowed {
 			allowed = true
 		}
@@ -240,7 +240,7 @@ func HandleV1DocumentRead(c echo.Context) error {
 	document := c.Param("document")
 
 	allowed := false
-	for bucket_allowed := range cfg.Server.Documents.Buckets {
+	for _, bucket_allowed := range cfg.Server.Documents.Buckets {
 		if bucket == bucket_allowed {
 			allowed = true
 		}
@@ -317,7 +317,7 @@ func HandleV1DocumentDelete(c echo.Context) error {
 	document := c.Param("document")
 
 	allowed := false
-	for bucket_allowed := range cfg.Server.Documents.Buckets {
+	for _, bucket_allowed := range cfg.Server.Documents.Buckets {
 		if bucket == bucket_allowed {
 			allowed = true
 		}

templates/html/admin-upload.gohtml

@@ -12,9 +12,8 @@
       <form class="form adminUploadForm" action="/admin/documents/upload" method="post" enctype="multipart/form-data">
         <label class="formLabel" for="bucket">Type de document:</label>
         <select class="formSelect" name="bucket" id="bucket">
-          {{ range .Buckets }}
+          <option class="formOption" value="proces-verbaux">Procès verbaux</option>
-            <option class="formOption" value="{{ .Name }}">{{ .DisplayName }}</option>
+          <option class="formOption" value="politiques-et-reglements">Politiques et Règlements</option>
-          {{ end }}
         </select>
         <br>
         Document: <input class="formDocUpload" type="file" name="document">

templates/html/documentation.gohtml

@@ -13,10 +13,8 @@
         <p>
           {{ range . }}
             {{ $bucket_name := .Name }}
-            {{ $bucket_display_name := .DisplayName }}
             <details>
-            <summary>{{ $bucket_display_name }}</summary>
+              <summary>{{ $bucket_name }}</summary>
-            
               <ul>
                 {{ range .Documents }}
                   <li> <a href="/public/documentation/{{ $bucket_name }}/{{ . }}">{{ . }}</a></li>