add transaction

formancehq · Jun 18, 2021 · 59c33d1 · 59c33d1
1 parent 45e8834
commit 59c33d1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/api/http.go b/api/http.go
@@ -52,8 +52,8 @@ func NewHttpAPI(lc fx.Lifecycle, l *ledger.Ledger) *HttpAPI {
 
 		err := l.Commit(t)
 
-		c.JSON(304, gin.H{
-			"ok": err != nil,
+		c.JSON(200, gin.H{
+			"ok": err == nil,
 		})
 	})
 

diff --git a/ledger/ledger.go b/ledger/ledger.go
@@ -57,6 +57,7 @@ func (l *Ledger) Close() {
 
 func (l *Ledger) Commit(t core.Transaction) error {
 	l.Lock()
+	defer l.Unlock()
 
 	count, _ := l.store.CountTransactions()
 	t.ID = count
@@ -134,8 +135,6 @@ func (l *Ledger) Commit(t core.Transaction) error {
 
 	l._last = &t
 
-	l.Unlock()
-
 	return err
 }