Merge pull request #189 from getAlby/fix/amt-validations

Fix/amt-validations
getAlby · Jun 10, 2022 · 53275f2 · 53275f2
2 parents 5a35d7c + 22a788e
commit 53275f2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/controllers/addinvoice.ctrl.go b/controllers/addinvoice.ctrl.go
@@ -61,7 +61,7 @@ func AddInvoice(c echo.Context, svc *service.LndhubService, userID int64) error
 	}
 
 	amount, err := svc.ParseInt(body.Amount)
-	if err != nil {
+	if err != nil || amount < 0 {
 		return c.JSON(http.StatusBadRequest, responses.BadArgumentsError)
 	}
 	c.Logger().Infof("Adding invoice: user_id:%v memo:%s value:%v description_hash:%s", userID, body.Memo, amount, body.DescriptionHash)

diff --git a/controllers/keysend.ctrl.go b/controllers/keysend.ctrl.go
@@ -24,7 +24,7 @@ func NewKeySendController(svc *service.LndhubService) *KeySendController {
 }
 
 type KeySendRequestBody struct {
-	Amount        int64             `json:"amount" validate:"required"`
+	Amount        int64             `json:"amount" validate:"required,gt=0"`
 	Destination   string            `json:"destination" validate:"required"`
 	Memo          string            `json:"memo" validate:"omitempty"`
 	CustomRecords map[string]string `json:"customRecords" validate:"omitempty"`