Add a failsafe if the user is created twice (It just happaned, this is
just to prevent further damages.)
This commit is contained in:
parent
97c28d7093
commit
f2d88296cb
@ -2,7 +2,7 @@ package pages
|
|||||||
|
|
||||||
import (
|
import (
|
||||||
"encoding/json"
|
"encoding/json"
|
||||||
"io/ioutil"
|
"io"
|
||||||
"net/http"
|
"net/http"
|
||||||
"net/url"
|
"net/url"
|
||||||
"os"
|
"os"
|
||||||
@ -75,7 +75,7 @@ func SignupPage(c *fiber.Ctx) error {
|
|||||||
}
|
}
|
||||||
defer resp.Body.Close()
|
defer resp.Body.Close()
|
||||||
|
|
||||||
bod, err := ioutil.ReadAll(resp.Body)
|
bod, err := io.ReadAll(resp.Body)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
log.Error("Error reading captcha response body", err)
|
log.Error("Error reading captcha response body", err)
|
||||||
}
|
}
|
||||||
@ -99,6 +99,18 @@ func SignupPage(c *fiber.Ctx) error {
|
|||||||
"status": c.Response().StatusCode(),
|
"status": c.Response().StatusCode(),
|
||||||
})
|
})
|
||||||
} else {
|
} else {
|
||||||
|
// Check if user already exists
|
||||||
|
// We'll check the home folder and see if the username folder exists.
|
||||||
|
_, err := os.Stat("/home/" + username)
|
||||||
|
if err == nil {
|
||||||
|
log.Error("User already exists", username)
|
||||||
|
return c.JSON(fiber.Map{
|
||||||
|
"username": username,
|
||||||
|
"message": "User already exists, Choose a different username.",
|
||||||
|
"status": c.Response().StatusCode(),
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
// create user file
|
// create user file
|
||||||
|
|
||||||
f, err := os.Create("/var/publapi/users/" + username + ".sh")
|
f, err := os.Create("/var/publapi/users/" + username + ".sh")
|
||||||
@ -150,4 +162,5 @@ func SignupPage(c *fiber.Ctx) error {
|
|||||||
"status": c.Response().StatusCode(),
|
"status": c.Response().StatusCode(),
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user