remove Atoi
isn't necessary anymore and sql injection still won't work
This commit is contained in:
parent
fb34c220b2
commit
ee9708be55
18
main.go
18
main.go
@ -7,7 +7,6 @@ import (
|
|||||||
"log"
|
"log"
|
||||||
"net/http"
|
"net/http"
|
||||||
"os"
|
"os"
|
||||||
"strconv"
|
|
||||||
|
|
||||||
"github.com/gin-gonic/gin"
|
"github.com/gin-gonic/gin"
|
||||||
"github.com/jackc/pgx/v5/pgxpool"
|
"github.com/jackc/pgx/v5/pgxpool"
|
||||||
@ -165,12 +164,7 @@ func listVideos(c *gin.Context) {
|
|||||||
|
|
||||||
func getVideo(c *gin.Context) {
|
func getVideo(c *gin.Context) {
|
||||||
var err error
|
var err error
|
||||||
inputId, err := strconv.Atoi(c.Param("id"))
|
rows, _ := dbpool.Query(context.Background(), "select filepath from videos where id = $1", c.Param("id"))
|
||||||
if err != nil {
|
|
||||||
log.Panicf("getVideo: %v\n", err)
|
|
||||||
}
|
|
||||||
|
|
||||||
rows, _ := dbpool.Query(context.Background(), "select filepath from videos where id = $1", inputId)
|
|
||||||
rows.Next()
|
rows.Next()
|
||||||
err = rows.Err()
|
err = rows.Err()
|
||||||
if err != nil {
|
if err != nil {
|
||||||
@ -189,12 +183,8 @@ func getVideo(c *gin.Context) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
func deleteVideo(c *gin.Context) {
|
func deleteVideo(c *gin.Context) {
|
||||||
inputId, err := strconv.Atoi(c.Param("id"))
|
var err error
|
||||||
if err != nil {
|
rows, _ := dbpool.Query(context.Background(), "select filepath from videos where id = $1", c.Param("id"))
|
||||||
log.Panicf("deleteVideo: %v\n", err)
|
|
||||||
}
|
|
||||||
|
|
||||||
rows, _ := dbpool.Query(context.Background(), "select filepath from videos where id = $1", inputId)
|
|
||||||
rows.Next()
|
rows.Next()
|
||||||
err = rows.Err()
|
err = rows.Err()
|
||||||
if err != nil {
|
if err != nil {
|
||||||
@ -209,7 +199,7 @@ func deleteVideo(c *gin.Context) {
|
|||||||
log.Panicf("deleteVideo: %v\n", err)
|
log.Panicf("deleteVideo: %v\n", err)
|
||||||
}
|
}
|
||||||
|
|
||||||
_, err = dbpool.Exec(context.Background(), "delete from videos where id = $1", inputId)
|
_, err = dbpool.Exec(context.Background(), "delete from videos where id = $1", c.Param("id"))
|
||||||
if err != nil {
|
if err != nil {
|
||||||
c.JSON(http.StatusInternalServerError, "Id was likely invalid")
|
c.JSON(http.StatusInternalServerError, "Id was likely invalid")
|
||||||
log.Panicf("deleteVideo: %v\n", err)
|
log.Panicf("deleteVideo: %v\n", err)
|
||||||
|
Loading…
Reference in New Issue
Block a user