Merge pull request #744 from ystia/backport40/bugfix/GH-741-log-outpu…

…t-handler-err [Backport 4.0 ] Yorc panics attempting to print an error handling a script execution stdout
ystia · May 25, 2021 · 32f91cd · 32f91cd
2 parents 305fe13 + 89a7aa9
commit 32f91cd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,7 @@
 
 ### BUG FIXES
 
+* Yorc panics attempting to print an error handling a script execution stdout ([GH-741](https://github.com/ystia/yorc/issues/741))
 * Error submitting a SLURM job with no execution option ([GH-739](https://github.com/ystia/yorc/issues/739))
 
 ## 4.0.7 (May 19, 2021)

diff --git a/prov/ansible/execution.go b/prov/ansible/execution.go
@@ -1270,7 +1270,7 @@ func (e *executionCommon) executePlaybook(ctx context.Context, retry bool,
 
 	err := cmd.Run()
 	if handlerErr := handler.stop(); handlerErr != nil {
-		log.Printf("Error stopping output handler: %s", err.Error())
+		log.Printf("Error stopping output handler: %s", handlerErr.Error())
 	}
 	if err != nil {
 		return e.checkAnsibleRetriableError(ctx, err)