[ 
https://issues.apache.org/jira/browse/TS-4498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15312498#comment-15312498
 ] 

ASF GitHub Bot commented on TS-4498:
------------------------------------

Github user jpeach commented on the issue:

    https://github.com/apache/trafficserver/pull/690
  
    Thanks @pbchou. I think we can make this a more complete fix. If you look 
at the way ``remap_load_plugin`` works, it it supposed to return non-zero on 
error and write the error message to ``errbuf``. Rather than emitting a 
``Warning`` in ``remap_load_plugin``, we should format the error into the 
output error buffer. Then the real error message will be available in 
``remap_parse_config_bti``, and we will emit the message correctly. 
Additionally, I think the ``Warning`` at ``MAP_ERROR`` and ``SignalError()`` 
should be ``Error``.
    
    Later we can look at improving this with 
[TS-4501](https://issues.apache.org/jira/browse/TS-4501).


> RemapConfig.cc - Print out error message on remap plugin init failure.
> ----------------------------------------------------------------------
>
>                 Key: TS-4498
>                 URL: https://issues.apache.org/jira/browse/TS-4498
>             Project: Traffic Server
>          Issue Type: Improvement
>          Components: Plugins
>            Reporter: Peter Chou
>
> Add printing of the returned error message to the Warning() if a remap plugin 
> fails to init. Currently it just says "bailing out" which is not as useful.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to