fix(scripting): use bg context for task execution

pkg/helper/context.go

@@ -0,0 +1,20 @@
+package helper
+
+import (
+	"context"
+	"time"
+
+	"github.com/ncarlier/readflow/pkg/constant"
+)
+
+// NewBackgroundContextWithValues create new context with same values than another context
+func NewBackgroundContextWithValues(src context.Context, timeout time.Duration) (context.Context, context.CancelFunc) {
+	ctx, cancel := context.WithTimeout(context.Background(), timeout)
+	ctx = context.WithValue(ctx, constant.ContextDownloader, src.Value(constant.ContextDownloader))
+	ctx = context.WithValue(ctx, constant.ContextIncomingWebhook, src.Value(constant.ContextIncomingWebhook))
+	ctx = context.WithValue(ctx, constant.ContextIsAdmin, src.Value(constant.ContextIsAdmin))
+	ctx = context.WithValue(ctx, constant.ContextRequestID, src.Value(constant.ContextRequestID))
+	ctx = context.WithValue(ctx, constant.ContextUser, src.Value(constant.ContextUser))
+	ctx = context.WithValue(ctx, constant.ContextUserID, src.Value(constant.ContextUserID))
+	return ctx, cancel
+}

pkg/service/articles_create.go

@@ -6,6 +6,7 @@ import (
 
 	"github.com/ncarlier/readflow/pkg/constant"
 	"github.com/ncarlier/readflow/pkg/event"
+	"github.com/ncarlier/readflow/pkg/helper"
 	"github.com/ncarlier/readflow/pkg/model"
 	"github.com/ncarlier/readflow/pkg/scripting"
 
@@ -102,8 +103,10 @@ func (reg *Registry) CreateArticle(ctx context.Context, form model.ArticleCreate
 	logger.Info().Uint("id", article.ID).Msg("article created")
 	// exec asynchronously other operations
 	go func() {
-		if err := reg.execOtherOperations(ctx, ops, article); err != nil {
-			logger.Info().Err(err).Msg(unableToCreateArticleErrorMsg)
+		bgCtx, cancel := helper.NewBackgroundContextWithValues(ctx, constant.DefaultTimeout)
+		defer cancel()
+		if err := reg.execOtherOperations(bgCtx, ops, article); err != nil {
+			logger.Info().Err(err).Msg("error while applying script operations")
 		}
 	}()
 	// emit article creation event