Browse Source

feat: duplicate site #80

0xJacky 2 years ago
parent
commit
a3dfc7af4a
3 changed files with 480 additions and 386 deletions
  1. 30 15
      frontend/src/views/domain/DomainList.vue
  2. 79 0
      frontend/src/views/domain/SiteDuplicate.vue
  3. 371 371
      server/api/domain.go

+ 30 - 15
frontend/src/views/domain/DomainList.vue

@@ -10,6 +10,7 @@ import domain from '@/api/domain'
 import {Badge, message} from 'ant-design-vue'
 import {h, ref} from 'vue'
 import {input} from '@/components/StdDataEntry'
+import SiteDuplicate from '@/views/domain/SiteDuplicate.vue'
 
 const columns = [{
     title: () => $gettext('Name'),
@@ -25,7 +26,7 @@ const columns = [{
     dataIndex: 'enabled',
     customRender: (args: customRender) => {
         const template: any = []
-        const {text, column} = args
+        const {text} = args
         if (text === true || text > 0) {
             template.push(<Badge status="success"/>)
             template.push($gettext('Enabled'))
@@ -83,6 +84,15 @@ function destroy(site_name: any) {
         message.error(e?.message ?? $gettext('Server error'))
     })
 }
+
+const show_duplicator = ref(false)
+
+const target = ref('')
+
+function handle_click_duplicate(name: string) {
+    show_duplicator.value = true
+    target.value = name
+}
 </script>
 
 <template>
@@ -99,25 +109,30 @@ function destroy(site_name: any) {
         >
             <template #actions="{record}">
                 <a-divider type="vertical"/>
-                <a v-if="record.enabled" @click="disable(record.name)">
+                <a-button type="link" size="small" v-if="record.enabled" @click="disable(record.name)">
                     {{ $gettext('Disabled') }}
-                </a>
-                <a v-else @click="enable(record.name)">
+                </a-button>
+                <a-button type="link" size="small" v-else @click="enable(record.name)">
                     {{ $gettext('Enabled') }}
-                </a>
-                <template v-if="!record.enabled">
-                    <a-divider type="vertical"/>
-                    <a-popconfirm
-                        :cancelText="$gettext('No')"
-                        :okText="$gettext('OK')"
-                        :title="$gettext('Are you sure you want to delete?')"
-                        @confirm="destroy(record['name'])">
-                        <a v-translate>Delete</a>
-                    </a-popconfirm>
-                </template>
+                </a-button>
+                <a-divider type="vertical"/>
+                <a-button type="link" size="small" @click="handle_click_duplicate(record.name)">
+                    {{ $gettext('Duplicate') }}
+                </a-button>
+                <a-divider type="vertical"/>
+                <a-popconfirm
+                    :cancelText="$gettext('No')"
+                    :okText="$gettext('OK')"
+                    :title="$gettext('Are you sure you want to delete?')"
+                    @confirm="destroy(record['name'])">
+                    <a-button type="link" size="small" :disabled="record.enabled">
+                        {{ $gettext('Delete') }}
+                    </a-button>
+                </a-popconfirm>
             </template>
         </std-table>
     </a-card>
+    <site-duplicate v-model:visible="show_duplicator" :name="target" @duplicated="table.get_list()"/>
 </template>
 
 <style scoped>

+ 79 - 0
frontend/src/views/domain/SiteDuplicate.vue

@@ -0,0 +1,79 @@
+<script setup lang="ts">
+import {computed, nextTick, reactive, ref, watch} from 'vue'
+import {useGettext} from 'vue3-gettext'
+import {Form, message} from 'ant-design-vue'
+import gettext from '@/gettext'
+import domain from '@/api/domain'
+
+const {$gettext} = useGettext()
+
+const props = defineProps(['visible', 'name'])
+const emit = defineEmits(['update:visible', 'duplicated'])
+
+const show = computed({
+    get() {
+        return props.visible
+    },
+    set(v) {
+        emit('update:visible', v)
+    }
+})
+
+const modelRef = reactive({name: ''})
+
+const rulesRef = reactive({
+    name: [
+        {
+            required: true,
+            message: () => $gettext('Please input name, ' +
+                'this will be used as the filename of the new configuration!')
+        }
+    ]
+})
+
+const {validate, validateInfos, clearValidate} = Form.useForm(modelRef, rulesRef)
+
+const loading = ref(false)
+
+function onSubmit() {
+    validate().then(async () => {
+        loading.value = true
+
+        domain.duplicate(props.name, {name: modelRef.name}).then(() => {
+            message.success($gettext('Duplicated successfully'))
+            show.value = false
+            emit('duplicated')
+        }).catch((e: any) => {
+            message.error($gettext(e?.message ?? 'Server error'))
+        })
+
+        loading.value = false
+    })
+}
+
+watch(() => props.visible, (v) => {
+    if (v) {
+        modelRef.name = ''
+        nextTick(() => clearValidate())
+    }
+})
+
+watch(() => gettext.current, () => {
+    clearValidate()
+})
+</script>
+
+<template>
+    <a-modal :title="$gettext('Duplicate')" v-model:visible="show" @ok="onSubmit"
+             :confirm-loading="loading">
+        <a-form layout="vertical">
+            <a-form-item :label="$gettext('Name')" v-bind="validateInfos.name">
+                <a-input v-model:value="modelRef.name"/>
+            </a-form-item>
+        </a-form>
+    </a-modal>
+</template>
+
+<style lang="less" scoped>
+
+</style>

+ 371 - 371
server/api/domain.go

@@ -1,435 +1,435 @@
 package api
 
 import (
-	"github.com/0xJacky/Nginx-UI/server/model"
-	"github.com/0xJacky/Nginx-UI/server/pkg/cert"
-	"github.com/0xJacky/Nginx-UI/server/pkg/config_list"
-	"github.com/0xJacky/Nginx-UI/server/pkg/helper"
-	"github.com/0xJacky/Nginx-UI/server/pkg/nginx"
-	"github.com/gin-gonic/gin"
-	"log"
-	"net/http"
-	"os"
-	"strings"
-	"time"
+    "github.com/0xJacky/Nginx-UI/server/model"
+    "github.com/0xJacky/Nginx-UI/server/pkg/cert"
+    "github.com/0xJacky/Nginx-UI/server/pkg/config_list"
+    "github.com/0xJacky/Nginx-UI/server/pkg/helper"
+    "github.com/0xJacky/Nginx-UI/server/pkg/nginx"
+    "github.com/gin-gonic/gin"
+    "log"
+    "net/http"
+    "os"
+    "strings"
+    "time"
 )
 
 func GetDomains(c *gin.Context) {
-	name := c.Query("name")
-	orderBy := c.Query("order_by")
-	sort := c.DefaultQuery("sort", "desc")
-
-	mySort := map[string]string{
-		"enabled": "bool",
-		"name":    "string",
-		"modify":  "time",
-	}
-
-	configFiles, err := os.ReadDir(nginx.GetConfPath("sites-available"))
-
-	if err != nil {
-		ErrHandler(c, err)
-		return
-	}
-
-	enabledConfig, err := os.ReadDir(nginx.GetConfPath("sites-enabled"))
-
-	if err != nil {
-		ErrHandler(c, err)
-		return
-	}
-
-	enabledConfigMap := make(map[string]bool)
-	for i := range enabledConfig {
-		enabledConfigMap[enabledConfig[i].Name()] = true
-	}
-
-	var configs []gin.H
-
-	for i := range configFiles {
-		file := configFiles[i]
-		fileInfo, _ := file.Info()
-		if !file.IsDir() {
-			if name != "" && !strings.Contains(file.Name(), name) {
-				continue
-			}
-			configs = append(configs, gin.H{
-				"name":    file.Name(),
-				"size":    fileInfo.Size(),
-				"modify":  fileInfo.ModTime(),
-				"enabled": enabledConfigMap[file.Name()],
-			})
-		}
-	}
-
-	configs = config_list.Sort(orderBy, sort, mySort[orderBy], configs)
-
-	c.JSON(http.StatusOK, gin.H{
-		"data": configs,
-	})
+    name := c.Query("name")
+    orderBy := c.Query("order_by")
+    sort := c.DefaultQuery("sort", "desc")
+
+    mySort := map[string]string{
+        "enabled": "bool",
+        "name":    "string",
+        "modify":  "time",
+    }
+
+    configFiles, err := os.ReadDir(nginx.GetConfPath("sites-available"))
+
+    if err != nil {
+        ErrHandler(c, err)
+        return
+    }
+
+    enabledConfig, err := os.ReadDir(nginx.GetConfPath("sites-enabled"))
+
+    if err != nil {
+        ErrHandler(c, err)
+        return
+    }
+
+    enabledConfigMap := make(map[string]bool)
+    for i := range enabledConfig {
+        enabledConfigMap[enabledConfig[i].Name()] = true
+    }
+
+    var configs []gin.H
+
+    for i := range configFiles {
+        file := configFiles[i]
+        fileInfo, _ := file.Info()
+        if !file.IsDir() {
+            if name != "" && !strings.Contains(file.Name(), name) {
+                continue
+            }
+            configs = append(configs, gin.H{
+                "name":    file.Name(),
+                "size":    fileInfo.Size(),
+                "modify":  fileInfo.ModTime(),
+                "enabled": enabledConfigMap[file.Name()],
+            })
+        }
+    }
+
+    configs = config_list.Sort(orderBy, sort, mySort[orderBy], configs)
+
+    c.JSON(http.StatusOK, gin.H{
+        "data": configs,
+    })
 }
 
 type CertificateInfo struct {
-	SubjectName string    `json:"subject_name"`
-	IssuerName  string    `json:"issuer_name"`
-	NotAfter    time.Time `json:"not_after"`
-	NotBefore   time.Time `json:"not_before"`
+    SubjectName string    `json:"subject_name"`
+    IssuerName  string    `json:"issuer_name"`
+    NotAfter    time.Time `json:"not_after"`
+    NotBefore   time.Time `json:"not_before"`
 }
 
 func GetDomain(c *gin.Context) {
-	rewriteName, ok := c.Get("rewriteConfigFileName")
+    rewriteName, ok := c.Get("rewriteConfigFileName")
 
-	name := c.Param("name")
+    name := c.Param("name")
 
-	// for modify filename
-	if ok {
-		name = rewriteName.(string)
-	}
+    // for modify filename
+    if ok {
+        name = rewriteName.(string)
+    }
 
-	path := nginx.GetConfPath("sites-available", name)
+    path := nginx.GetConfPath("sites-available", name)
 
-	enabled := true
-	if _, err := os.Stat(nginx.GetConfPath("sites-enabled", name)); os.IsNotExist(err) {
-		enabled = false
-	}
+    enabled := true
+    if _, err := os.Stat(nginx.GetConfPath("sites-enabled", name)); os.IsNotExist(err) {
+        enabled = false
+    }
 
-	c.Set("maybe_error", "nginx_config_syntax_error")
-	config, err := nginx.ParseNgxConfig(path)
+    c.Set("maybe_error", "nginx_config_syntax_error")
+    config, err := nginx.ParseNgxConfig(path)
 
-	if err != nil {
-		ErrHandler(c, err)
-		return
-	}
+    if err != nil {
+        ErrHandler(c, err)
+        return
+    }
 
-	c.Set("maybe_error", "")
+    c.Set("maybe_error", "")
 
-	certInfoMap := make(map[int]CertificateInfo)
-	var serverName string
-	for serverIdx, server := range config.Servers {
-		for _, directive := range server.Directives {
+    certInfoMap := make(map[int]CertificateInfo)
+    var serverName string
+    for serverIdx, server := range config.Servers {
+        for _, directive := range server.Directives {
 
-			if directive.Directive == "server_name" {
-				serverName = strings.ReplaceAll(directive.Params, " ", "_")
-				continue
-			}
+            if directive.Directive == "server_name" {
+                serverName = strings.ReplaceAll(directive.Params, " ", "_")
+                continue
+            }
 
-			if directive.Directive == "ssl_certificate" {
+            if directive.Directive == "ssl_certificate" {
 
-				pubKey, err := cert.GetCertInfo(directive.Params)
+                pubKey, err := cert.GetCertInfo(directive.Params)
 
-				if err != nil {
-					log.Println("Failed to get certificate information", err)
-					break
-				}
+                if err != nil {
+                    log.Println("Failed to get certificate information", err)
+                    break
+                }
 
-				certInfoMap[serverIdx] = CertificateInfo{
-					SubjectName: pubKey.Subject.CommonName,
-					IssuerName:  pubKey.Issuer.CommonName,
-					NotAfter:    pubKey.NotAfter,
-					NotBefore:   pubKey.NotBefore,
-				}
+                certInfoMap[serverIdx] = CertificateInfo{
+                    SubjectName: pubKey.Subject.CommonName,
+                    IssuerName:  pubKey.Issuer.CommonName,
+                    NotAfter:    pubKey.NotAfter,
+                    NotBefore:   pubKey.NotBefore,
+                }
 
-				break
-			}
-		}
-	}
+                break
+            }
+        }
+    }
 
-	certModel, _ := model.FirstCert(serverName)
+    certModel, _ := model.FirstCert(serverName)
 
-	c.Set("maybe_error", "nginx_config_syntax_error")
+    c.Set("maybe_error", "nginx_config_syntax_error")
 
-	c.JSON(http.StatusOK, gin.H{
-		"enabled":   enabled,
-		"name":      name,
-		"config":    config.FmtCode(),
-		"tokenized": config,
-		"auto_cert": certModel.AutoCert == model.AutoCertEnabled,
-		"cert_info": certInfoMap,
-	})
+    c.JSON(http.StatusOK, gin.H{
+        "enabled":   enabled,
+        "name":      name,
+        "config":    config.FmtCode(),
+        "tokenized": config,
+        "auto_cert": certModel.AutoCert == model.AutoCertEnabled,
+        "cert_info": certInfoMap,
+    })
 
 }
 
 func SaveDomain(c *gin.Context) {
-	name := c.Param("name")
-
-	if name == "" {
-		c.JSON(http.StatusNotAcceptable, gin.H{
-			"message": "param name is empty",
-		})
-		return
-	}
-
-	var json struct {
-		Name      string `json:"name" binding:"required"`
-		Content   string `json:"content" binding:"required"`
-		Overwrite bool   `json:"overwrite"`
-	}
-
-	if !BindAndValid(c, &json) {
-		return
-	}
-
-	path := nginx.GetConfPath("sites-available", name)
-
-	if !json.Overwrite && helper.FileExists(path) {
-		c.JSON(http.StatusNotAcceptable, gin.H{
-			"message": "File exists",
-		})
-		return
-	}
-
-	err := os.WriteFile(path, []byte(json.Content), 0644)
-	if err != nil {
-		ErrHandler(c, err)
-		return
-	}
-	enabledConfigFilePath := nginx.GetConfPath("sites-enabled", name)
-	// rename the config file if needed
-	if name != json.Name {
-		newPath := nginx.GetConfPath("sites-available", json.Name)
-		// check if dst file exists, do not rename
-		if helper.FileExists(newPath) {
-			c.JSON(http.StatusNotAcceptable, gin.H{
-				"message": "File exists",
-			})
-			return
-		}
-		// recreate soft link
-		if helper.FileExists(enabledConfigFilePath) {
-			_ = os.Remove(enabledConfigFilePath)
-			enabledConfigFilePath = nginx.GetConfPath("sites-enabled", json.Name)
-			err = os.Symlink(newPath, enabledConfigFilePath)
-
-			if err != nil {
-				ErrHandler(c, err)
-				return
-			}
-		}
-
-		err = os.Rename(path, newPath)
-		if err != nil {
-			ErrHandler(c, err)
-			return
-		}
-
-		name = json.Name
-		c.Set("rewriteConfigFileName", name)
-	}
-
-	enabledConfigFilePath = nginx.GetConfPath("sites-enabled", name)
-	if helper.FileExists(enabledConfigFilePath) {
-		// Test nginx configuration
-		output := nginx.TestConf()
-		if nginx.GetLogLevel(output) >= nginx.Warn {
-			c.JSON(http.StatusInternalServerError, gin.H{
-				"message": output,
-				"error":   "nginx_config_syntax_error",
-			})
-			return
-		}
-
-		output = nginx.Reload()
-
-		if nginx.GetLogLevel(output) >= nginx.Warn {
-			c.JSON(http.StatusInternalServerError, gin.H{
-				"message": output,
-			})
-			return
-		}
-	}
-
-	GetDomain(c)
+    name := c.Param("name")
+
+    if name == "" {
+        c.JSON(http.StatusNotAcceptable, gin.H{
+            "message": "param name is empty",
+        })
+        return
+    }
+
+    var json struct {
+        Name      string `json:"name" binding:"required"`
+        Content   string `json:"content" binding:"required"`
+        Overwrite bool   `json:"overwrite"`
+    }
+
+    if !BindAndValid(c, &json) {
+        return
+    }
+
+    path := nginx.GetConfPath("sites-available", name)
+
+    if !json.Overwrite && helper.FileExists(path) {
+        c.JSON(http.StatusNotAcceptable, gin.H{
+            "message": "File exists",
+        })
+        return
+    }
+
+    err := os.WriteFile(path, []byte(json.Content), 0644)
+    if err != nil {
+        ErrHandler(c, err)
+        return
+    }
+    enabledConfigFilePath := nginx.GetConfPath("sites-enabled", name)
+    // rename the config file if needed
+    if name != json.Name {
+        newPath := nginx.GetConfPath("sites-available", json.Name)
+        // check if dst file exists, do not rename
+        if helper.FileExists(newPath) {
+            c.JSON(http.StatusNotAcceptable, gin.H{
+                "message": "File exists",
+            })
+            return
+        }
+        // recreate soft link
+        if helper.FileExists(enabledConfigFilePath) {
+            _ = os.Remove(enabledConfigFilePath)
+            enabledConfigFilePath = nginx.GetConfPath("sites-enabled", json.Name)
+            err = os.Symlink(newPath, enabledConfigFilePath)
+
+            if err != nil {
+                ErrHandler(c, err)
+                return
+            }
+        }
+
+        err = os.Rename(path, newPath)
+        if err != nil {
+            ErrHandler(c, err)
+            return
+        }
+
+        name = json.Name
+        c.Set("rewriteConfigFileName", name)
+    }
+
+    enabledConfigFilePath = nginx.GetConfPath("sites-enabled", name)
+    if helper.FileExists(enabledConfigFilePath) {
+        // Test nginx configuration
+        output := nginx.TestConf()
+        if nginx.GetLogLevel(output) >= nginx.Warn {
+            c.JSON(http.StatusInternalServerError, gin.H{
+                "message": output,
+                "error":   "nginx_config_syntax_error",
+            })
+            return
+        }
+
+        output = nginx.Reload()
+
+        if nginx.GetLogLevel(output) >= nginx.Warn {
+            c.JSON(http.StatusInternalServerError, gin.H{
+                "message": output,
+            })
+            return
+        }
+    }
+
+    GetDomain(c)
 }
 
 func EnableDomain(c *gin.Context) {
-	configFilePath := nginx.GetConfPath("sites-available", c.Param("name"))
-	enabledConfigFilePath := nginx.GetConfPath("sites-enabled", c.Param("name"))
-
-	_, err := os.Stat(configFilePath)
-
-	if err != nil {
-		ErrHandler(c, err)
-		return
-	}
-
-	if _, err = os.Stat(enabledConfigFilePath); os.IsNotExist(err) {
-		err = os.Symlink(configFilePath, enabledConfigFilePath)
-
-		if err != nil {
-			ErrHandler(c, err)
-			return
-		}
-	}
-
-	// Test nginx config, if not pass then disable the site.
-	output := nginx.TestConf()
-
-	if nginx.GetLogLevel(output) >= nginx.Warn {
-		_ = os.Remove(enabledConfigFilePath)
-		c.JSON(http.StatusInternalServerError, gin.H{
-			"message": output,
-		})
-		return
-	}
-
-	output = nginx.Reload()
-
-	if nginx.GetLogLevel(output) >= nginx.Warn {
-		c.JSON(http.StatusInternalServerError, gin.H{
-			"message": output,
-		})
-		return
-	}
-
-	c.JSON(http.StatusOK, gin.H{
-		"message": "ok",
-	})
+    configFilePath := nginx.GetConfPath("sites-available", c.Param("name"))
+    enabledConfigFilePath := nginx.GetConfPath("sites-enabled", c.Param("name"))
+
+    _, err := os.Stat(configFilePath)
+
+    if err != nil {
+        ErrHandler(c, err)
+        return
+    }
+
+    if _, err = os.Stat(enabledConfigFilePath); os.IsNotExist(err) {
+        err = os.Symlink(configFilePath, enabledConfigFilePath)
+
+        if err != nil {
+            ErrHandler(c, err)
+            return
+        }
+    }
+
+    // Test nginx config, if not pass then disable the site.
+    output := nginx.TestConf()
+
+    if nginx.GetLogLevel(output) >= nginx.Warn {
+        _ = os.Remove(enabledConfigFilePath)
+        c.JSON(http.StatusInternalServerError, gin.H{
+            "message": output,
+        })
+        return
+    }
+
+    output = nginx.Reload()
+
+    if nginx.GetLogLevel(output) >= nginx.Warn {
+        c.JSON(http.StatusInternalServerError, gin.H{
+            "message": output,
+        })
+        return
+    }
+
+    c.JSON(http.StatusOK, gin.H{
+        "message": "ok",
+    })
 }
 
 func DisableDomain(c *gin.Context) {
-	enabledConfigFilePath := nginx.GetConfPath("sites-enabled", c.Param("name"))
-
-	_, err := os.Stat(enabledConfigFilePath)
-
-	if err != nil {
-		ErrHandler(c, err)
-		return
-	}
-
-	err = os.Remove(enabledConfigFilePath)
-
-	if err != nil {
-		ErrHandler(c, err)
-		return
-	}
-
-	// delete auto cert record
-	certModel := model.Cert{Domain: c.Param("name")}
-	err = certModel.Remove()
-	if err != nil {
-		ErrHandler(c, err)
-		return
-	}
-
-	output := nginx.Reload()
-
-	if nginx.GetLogLevel(output) >= nginx.Warn {
-		c.JSON(http.StatusInternalServerError, gin.H{
-			"message": output,
-		})
-		return
-	}
-
-	c.JSON(http.StatusOK, gin.H{
-		"message": "ok",
-	})
+    enabledConfigFilePath := nginx.GetConfPath("sites-enabled", c.Param("name"))
+
+    _, err := os.Stat(enabledConfigFilePath)
+
+    if err != nil {
+        ErrHandler(c, err)
+        return
+    }
+
+    err = os.Remove(enabledConfigFilePath)
+
+    if err != nil {
+        ErrHandler(c, err)
+        return
+    }
+
+    // delete auto cert record
+    certModel := model.Cert{Domain: c.Param("name")}
+    err = certModel.Remove()
+    if err != nil {
+        ErrHandler(c, err)
+        return
+    }
+
+    output := nginx.Reload()
+
+    if nginx.GetLogLevel(output) >= nginx.Warn {
+        c.JSON(http.StatusInternalServerError, gin.H{
+            "message": output,
+        })
+        return
+    }
+
+    c.JSON(http.StatusOK, gin.H{
+        "message": "ok",
+    })
 }
 
 func DeleteDomain(c *gin.Context) {
-	var err error
-	name := c.Param("name")
-	availablePath := nginx.GetConfPath("sites-available", name)
-	enabledPath := nginx.GetConfPath("sites-enabled", name)
-
-	if _, err = os.Stat(availablePath); os.IsNotExist(err) {
-		c.JSON(http.StatusNotFound, gin.H{
-			"message": "site not found",
-		})
-		return
-	}
-
-	if _, err = os.Stat(enabledPath); err == nil {
-		c.JSON(http.StatusNotAcceptable, gin.H{
-			"message": "site is enabled",
-		})
-		return
-	}
-
-	certModel := model.Cert{Domain: name}
-	_ = certModel.Remove()
-
-	err = os.Remove(availablePath)
-
-	if err != nil {
-		ErrHandler(c, err)
-		return
-	}
-
-	c.JSON(http.StatusOK, gin.H{
-		"message": "ok",
-	})
+    var err error
+    name := c.Param("name")
+    availablePath := nginx.GetConfPath("sites-available", name)
+    enabledPath := nginx.GetConfPath("sites-enabled", name)
+
+    if _, err = os.Stat(availablePath); os.IsNotExist(err) {
+        c.JSON(http.StatusNotFound, gin.H{
+            "message": "site not found",
+        })
+        return
+    }
+
+    if _, err = os.Stat(enabledPath); err == nil {
+        c.JSON(http.StatusNotAcceptable, gin.H{
+            "message": "site is enabled",
+        })
+        return
+    }
+
+    certModel := model.Cert{Domain: name}
+    _ = certModel.Remove()
+
+    err = os.Remove(availablePath)
+
+    if err != nil {
+        ErrHandler(c, err)
+        return
+    }
+
+    c.JSON(http.StatusOK, gin.H{
+        "message": "ok",
+    })
 
 }
 
 func AddDomainToAutoCert(c *gin.Context) {
-	domain := c.Param("domain")
-	domain = strings.ReplaceAll(domain, " ", "_")
-	certModel, err := model.FirstOrCreateCert(domain)
+    domain := c.Param("domain")
+    domain = strings.ReplaceAll(domain, " ", "_")
+    certModel, err := model.FirstOrCreateCert(domain)
 
-	if err != nil {
-		ErrHandler(c, err)
-		return
-	}
+    if err != nil {
+        ErrHandler(c, err)
+        return
+    }
 
-	err = certModel.Updates(&model.Cert{
-		AutoCert: model.AutoCertEnabled,
-	})
+    err = certModel.Updates(&model.Cert{
+        AutoCert: model.AutoCertEnabled,
+    })
 
-	if err != nil {
-		ErrHandler(c, err)
-		return
-	}
+    if err != nil {
+        ErrHandler(c, err)
+        return
+    }
 
-	c.JSON(http.StatusOK, certModel)
+    c.JSON(http.StatusOK, certModel)
 }
 
 func RemoveDomainFromAutoCert(c *gin.Context) {
-	domain := c.Param("domain")
-	domain = strings.ReplaceAll(domain, " ", "_")
-	certModel := model.Cert{
-		Domain: domain,
-	}
-
-	err := certModel.Updates(&model.Cert{
-		AutoCert: model.AutoCertDisabled,
-	})
-
-	if err != nil {
-		ErrHandler(c, err)
-		return
-	}
-	c.JSON(http.StatusOK, nil)
+    domain := c.Param("domain")
+    domain = strings.ReplaceAll(domain, " ", "_")
+    certModel := model.Cert{
+        Domain: domain,
+    }
+
+    err := certModel.Updates(&model.Cert{
+        AutoCert: model.AutoCertDisabled,
+    })
+
+    if err != nil {
+        ErrHandler(c, err)
+        return
+    }
+    c.JSON(http.StatusOK, nil)
 }
 
 func DuplicateSite(c *gin.Context) {
-	name := c.Param("name")
+    name := c.Param("name")
 
-	var json struct {
-		Name string `json:"name"`
-	}
+    var json struct {
+        Name string `json:"name" binding:"required"`
+    }
 
-	if !BindAndValid(c, &json) {
-		return
-	}
+    if !BindAndValid(c, &json) {
+        return
+    }
 
-	src := nginx.GetConfPath("sites-available", name)
-	dst := nginx.GetConfPath("sites-available", json.Name)
+    src := nginx.GetConfPath("sites-available", name)
+    dst := nginx.GetConfPath("sites-available", json.Name)
 
-	if helper.FileExists(dst) {
-		c.JSON(http.StatusNotAcceptable, gin.H{
-			"message": "file exists",
-		})
-		return
-	}
+    if helper.FileExists(dst) {
+        c.JSON(http.StatusNotAcceptable, gin.H{
+            "message": "File exists",
+        })
+        return
+    }
 
-	_, err := helper.CopyFile(src, dst)
+    _, err := helper.CopyFile(src, dst)
 
-	if err != nil {
-		ErrHandler(c, err)
-		return
-	}
+    if err != nil {
+        ErrHandler(c, err)
+        return
+    }
 
-	c.JSON(http.StatusOK, gin.H{
-		"dst": dst,
-	})
+    c.JSON(http.StatusOK, gin.H{
+        "dst": dst,
+    })
 }