fix: leaked connections

Creating a transaction and failing to end it with either a commit or
rollback leaves db pool connections in an acquired state, unavailable to
other requests. There were two instances where this was happening, this
resulted in the service eventually becoming unresponsive after the
offending code was called enough times and all the connections were
exhausted.

internal/service/bouncer.go

@@ -17,13 +17,7 @@ func (b *Bouncer) Check(ctx context.Context, endpoint, dialed string) model.Boun
 		Allow: false,
 	}
 
-	tx, err := b.Begin(ctx)
-	if err != nil {
-		slog.Error("Unable to start transaction", slog.String("reason", err.Error()))
-		return result
-	}
-
-	queries := sqlc.New(tx)
+	queries := sqlc.New(b.Cursor)
 	row, err := queries.GetEndpointByExtension(ctx, sqlc.GetEndpointByExtensionParams{
 		ID:        endpoint,
 		Extension: db.Text(dialed),

internal/service/endpoint.go

@@ -156,6 +156,7 @@ func (e *EndpointService) Update(ctx context.Context, sid int32, payload model.P
 	if err != nil {
 		return model.Endpoint{}, err
 	}
+	defer tx.Rollback(ctx)
 
 	queries := sqlc.New(tx)
 	endpoint, err := queries.GetEndpointByID(ctx, sid)