fix: play panic recover (#238)

* chore: ignore node_modules in go

* chore: ignore node_modules in go

* fix: don't panic in pimo play

* fix: lint

* docs: fix changelog

* Update internal/app/pimo/play.go

Co-authored-by: Youen Péron <[email protected]>

---------

Co-authored-by: Youen Péron <[email protected]>

Author: adrienaury

CHANGELOG.md

@@ -17,7 +17,7 @@ Types of changes
 ## [1.18.1]
 
 - `Fixed` local seed parameter should be consistent between different jsonpath
-- `Fixed` PIMO Play web assembly version handle panics and recover gracefully when panics occure
+- `Fixed` PIMO Play handle panics and recover gracefully when panics occure
 
 ## [1.18.0]
 

internal/app/pimo/play.go

@@ -46,6 +46,15 @@ func Play(enableSecurity bool) *echo.Echo {
 
 // play binds client interface data entry to be processed and returns the transformed json to client
 func play(ctx echo.Context) error {
+	// dont't panic
+	defer func() error {
+		if r := recover(); r != nil {
+			log.Error().AnErr("panic", r.(error)).Msg("Recovering from panic in play.")
+			return ctx.String(http.StatusInternalServerError, r.(error).Error())
+		}
+		return nil
+	}() //nolint:errcheck
+
 	config := Config{
 		EmptyInput:       false,
 		RepeatUntil:      "",
@@ -110,6 +119,15 @@ func play(ctx echo.Context) error {
 }
 
 func flowchart(ctx echo.Context) error {
+	// dont't panic
+	defer func() error {
+		if r := recover(); r != nil {
+			log.Error().AnErr("panic", r.(error)).Msg("Recovering from panic in flow.")
+			return ctx.String(http.StatusInternalServerError, r.(error).Error())
+		}
+		return nil
+	}() //nolint:errcheck
+
 	var dataInput map[string]interface{}
 
 	err := ctx.Bind(&dataInput)

web/play/.gitignore

@@ -2,6 +2,7 @@
 
 # dependencies
 /node_modules
+!/node_modules/go.mod
 /.pnp
 .pnp.js
 
@@ -23,6 +24,6 @@ npm-debug.log*
 yarn-debug.log*
 yarn-error.log*
 
-# Elm 
+# Elm
 elm-stuff/
-gen/
+gen/