[bookkeeper] branch master updated: ISSUE #1687: Using eval instead of exec to handle quotes in healtcheck.sh

[sijie]

This is an automated email from the ASF dual-hosted git repository.

sijie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git


The following commit(s) were added to refs/heads/master by this push:
     new 0284c14  ISSUE #1687: Using eval instead of exec to handle quotes in 
healtcheck.sh
0284c14 is described below

commit 0284c147d5a81578b7005af6f8ee5bd28c6cae15
Author: Mate Varga <m...@matevarga.net>
AuthorDate: Thu Sep 20 10:08:33 2018 +0200

    ISSUE #1687: Using eval instead of exec to handle quotes in healtcheck.sh
    
    Descriptions of the changes in this PR:
    
    Fixes https://github.com/apache/bookkeeper/issues/1687
    
    
    ### Motivation
    
    https://github.com/apache/bookkeeper/issues/1687
    
    ### Changes
    Using eval instead of exec to run the health check script.
    
    Master Issue: #1687
    
    
    
    
    Author: matevarga <m...@monadic.hu>
    
    Reviewers: Sijie Guo <si...@apache.org>, Enrico Olivelli 
<eolive...@gmail.com>
    
    This closes #1688 from matevarga/master, closes #1687
---
 docker/scripts/healthcheck.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/docker/scripts/healthcheck.sh b/docker/scripts/healthcheck.sh
index 47a21bf..943be26 100755
--- a/docker/scripts/healthcheck.sh
+++ b/docker/scripts/healthcheck.sh
@@ -31,4 +31,4 @@ DEFAULT_HEALTH_CHECK_CMD="/opt/bookkeeper/bin/bookkeeper 
shell bookiesanity"
 
 HEALTH_CHECK_CMD=${HEALTH_CHECK_CMD:-"${DEFAULT_HEALTH_CHECK_CMD}"}
 
-exec "${HEALTH_CHECK_CMD}"
+eval "${HEALTH_CHECK_CMD}"