Fix Arabic Selection Option

.gitignore

@@ -22,3 +22,4 @@ cover.out
 coverage.tmp
 tsconfig.tsbuildinfo
 etc/*.pem
+docker-compose.yml

Dockerfile

@@ -34,8 +34,10 @@ RUN make build-ui
 ################
 FROM --platform=${TARGETPLATFORM:-linux/amd64} debian:bookworm-slim
 
-RUN apt-get update
-RUN apt-get install -y ca-certificates
+RUN apt-get update && \
+    apt-get install -y ca-certificates && \
+    apt-get clean && \
+    rm -rf /var/lib/apt/lists/*
 
 WORKDIR /app
 

app/actions/tenant.go

@@ -144,7 +144,7 @@ func (action *UpdateTenantSettings) Validate(ctx context.Context, user *entity.U
 		MinHeight:    200,
 		MinWidth:     200,
 		MaxKilobytes: 100,
-		ExactRatio:   true,
+		ExactRatio:   false,
 	})
 	if err != nil {
 		return validate.Error(err)

app/handlers/images.go

@@ -6,21 +6,20 @@ import (
 	"image/color"
 	"image/png"
 	"net/http"
+	"net/url"
 	"os"
 	"strings"
 	"time"
-
 	"github.com/getfider/fider/app/models/cmd"
 	"github.com/getfider/fider/app/models/dto"
 	"github.com/getfider/fider/app/models/query"
-
 	"github.com/getfider/fider/app/pkg/bus"
 	"github.com/getfider/fider/app/pkg/crypto"
 	"github.com/getfider/fider/app/pkg/env"
+	"github.com/getfider/fider/app/pkg/letteravatar"
 	"github.com/getfider/fider/app/pkg/log"
 	"github.com/getfider/fider/app/pkg/web"
 	"github.com/goenning/imagic"
-	"github.com/goenning/letteravatar"
 )
 
 // LetterAvatar returns a letter gravatar picture based on given name
@@ -32,22 +31,38 @@ func LetterAvatar() web.HandlerFunc {
 			name = "?"
 		}
 
+		// URL decode the name
+		decodedName, err := url.QueryUnescape(name)
+		if err != nil {
+			log.Error(c, err)
+			return c.Failure(err)
+		}
+
 		size, err := c.QueryParamAsInt("size")
 		if err != nil {
+			log.Error(c, err)
 			return c.BadRequest(web.Map{})
 		}
 		size = between(size, 50, 200)
 
-		img, err := letteravatar.Draw(size, strings.ToUpper(letteravatar.Extract(name)), &letteravatar.Options{
-			PaletteKey: fmt.Sprintf("%s:%s", id, name),
+		extractedLetter := letteravatar.Extract(decodedName)
+		log.Debugf(c, "Generating letter avatar. Name: @{Name}, Letter: @{Letter}", dto.Props{
+			"Name":   decodedName,
+			"Letter": extractedLetter,
+		})
+
+		img, err := letteravatar.Draw(size, extractedLetter, &letteravatar.Options{
+			PaletteKey: fmt.Sprintf("%s:%s", id, decodedName),
 		})
 		if err != nil {
+			log.Error(c, err)
 			return c.Failure(err)
 		}
 
 		buf := new(bytes.Buffer)
 		err = png.Encode(buf, img)
 		if err != nil {
+			log.Error(c, err)
 			return c.Failure(err)
 		}
 
@@ -70,10 +85,12 @@ func Gravatar() web.HandlerFunc {
 
 		size = between(size, 50, 200)
 
-		if err == nil && id > 0 {
+		var userName string
+		if id > 0 {
 			userByID := &query.GetUserByID{UserID: id}
 			err := bus.Dispatch(c, userByID)
 			if err == nil && userByID.Result.Tenant.ID == c.Tenant().ID {
+				userName = userByID.Result.Name
 				if userByID.Result.Email != "" {
 					url := fmt.Sprintf("https://www.gravatar.com/avatar/%s?s=%d&d=404", crypto.MD5(strings.ToLower(userByID.Result.Email)), size)
 					cacheKey := fmt.Sprintf("gravatar:%s", url)
@@ -105,7 +122,44 @@ func Gravatar() web.HandlerFunc {
 			}
 		}
 
-		return LetterAvatar()(c)
+		// Fallback to letter avatar if:
+		// 1. User ID is invalid
+		// 2. User not found
+		// 3. User has no email
+		// 4. Gravatar not found
+		name := c.QueryParam("name")
+		if name == "" {
+			name = userName
+		}
+		if name == "" {
+			name = "?"
+		}
+
+		// URL decode the name (matching LetterAvatar behavior)
+		decodedName, err := url.QueryUnescape(name)
+		if err != nil {
+			log.Error(c, err)
+			return c.Failure(err)
+		}
+
+		// Use the same mechanism as LetterAvatar
+		extractedLetter := letteravatar.Extract(decodedName)
+		img, err := letteravatar.Draw(size, extractedLetter, &letteravatar.Options{
+			PaletteKey: fmt.Sprintf("%s:%s", fmt.Sprint(id), decodedName),
+		})
+		if err != nil {
+			log.Error(c, err)
+			return c.Failure(err)
+		}
+
+		buf := new(bytes.Buffer)
+		err = png.Encode(buf, img)
+		if err != nil {
+			log.Error(c, err)
+			return c.Failure(err)
+		}
+
+		return c.Image("image/png", buf.Bytes())
 	}
 }
 
@@ -190,3 +244,21 @@ func ViewUploadedImage() web.HandlerFunc {
 		return c.Image(q.Result.ContentType, bytes)
 	}
 }
+
+// ValidateLogoUpload validates if logo upload is within allowed formats and size
+func ValidateLogoUpload(c *web.Context) error {
+	contentType := c.Request.GetHeader("Content-Type")
+	if contentType != "image/png" && contentType != "image/jpeg" && contentType != "image/gif" && contentType != "image/svg+xml" {
+		return c.BadRequest(web.Map{
+			"message": "File format not supported. Upload only JPG, GIF, PNG or SVG files.",
+		})
+	}
+
+	if c.Request.ContentLength > 100*1024 {
+		return c.BadRequest(web.Map{
+			"message": "File size is too large. Maximum allowed size is 100KB.",
+		})
+	}
+
+	return nil
+}

app/pkg/i18n/i18n.go

@@ -29,6 +29,7 @@ var localeToPlurals = map[string]string{
 	"it":    "it",
 	"tr":    "tr",
 	"zh-CN": "zh",
+	"ar":    "ar",
 }
 
 type Params map[string]any