Return error if internal error occurs (#301)

* fix cases of not returning error through InternalError
ReconfigureIO · Nov 8, 2018 · b8f02dc · b8f02dc
1 parent d193c3c
commit b8f02dc
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/handlers/api/build.go b/handlers/api/build.go
@@ -277,7 +277,7 @@ func (b Build) CreateEvent(c *gin.Context) {
 	}
 
 	if err != nil {
-		sugar.InternalError(c, nil)
+		sugar.InternalError(c, err)
 		return
 	}
 	eventMessage := "Build entered state:" + event.Status

diff --git a/handlers/api/deployment.go b/handlers/api/deployment.go
@@ -269,7 +269,7 @@ func (d Deployment) CreateEvent(c *gin.Context) {
 	newEvent, err := d.AddEvent(c, dep, event)
 
 	if err != nil {
-		sugar.InternalError(c, nil)
+		sugar.InternalError(c, err)
 		return
 	}
 

diff --git a/handlers/api/graph.go b/handlers/api/graph.go
@@ -241,7 +241,7 @@ func (g Graph) CreateEvent(c *gin.Context) {
 	newEvent, err := BatchService{AWS: g.AWS}.AddEvent(&graph.BatchJob, event)
 
 	if err != nil {
-		sugar.InternalError(c, nil)
+		sugar.InternalError(c, err)
 		return
 	}
 

diff --git a/handlers/api/simulation.go b/handlers/api/simulation.go
@@ -226,7 +226,7 @@ func (s Simulation) CreateEvent(c *gin.Context) {
 	newEvent, err := BatchService{AWS: s.AWS}.AddEvent(&sim.BatchJob, event)
 
 	if err != nil {
-		sugar.InternalError(c, nil)
+		sugar.InternalError(c, err)
 		return
 	}