From c202911fc830bb5d485d224dbca781b58ff2bd7c Mon Sep 17 00:00:00 2001
From: hexxa <hexxa@outlook.com>
Date: Sat, 16 Apr 2022 11:12:01 +0800
Subject: [PATCH] fix: check length of site name and site description

---
 src/client/web/src/components/topbar.tsx | 1 +
 src/handlers/settings/handlers.go        | 6 ++++--
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/client/web/src/components/topbar.tsx b/src/client/web/src/components/topbar.tsx
index 4c522a9..6d057ed 100644
--- a/src/client/web/src/components/topbar.tsx
+++ b/src/client/web/src/components/topbar.tsx
@@ -63,6 +63,7 @@ export class TopBar extends React.Component<Props, State, {}> {
               href="https://github.com/ihexxa/quickshare"
               target="_blank"
               className="h5 bold"
+              title={this.props.ui.clientCfg.siteDesc}
             >
               {this.props.ui.clientCfg.siteName}
             </a>,
diff --git a/src/handlers/settings/handlers.go b/src/handlers/settings/handlers.go
index 4df163b..a2082f3 100644
--- a/src/handlers/settings/handlers.go
+++ b/src/handlers/settings/handlers.go
@@ -83,8 +83,10 @@ func (h *SettingsSvc) SetClientCfg(c *gin.Context) {
 }
 
 func validateClientCfg(cfg *db.ClientConfig) error {
-	if cfg.SiteName == "" {
-		return errors.New("site name is empty")
+	if len(cfg.SiteName) == 0 || len(cfg.SiteName) >= 12 {
+		return errors.New("site name is too short or too long")
+	} else if len(cfg.SiteDesc) >= 64 {
+		return errors.New("site description is too short or too long")
 	}
 	return nil
 }