You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2022/11/21 10:39:37 UTC
[GitHub] [trafficserver] brbzull0 opened a new pull request, #9212: traffic_ctl - Fix number of mandatory arguments when sending a plugin msg.
brbzull0 opened a new pull request, #9212:
URL: https://github.com/apache/trafficserver/pull/9212
Fix number of mandatory arguments when sending a plugin msg.
It looks like this got badly merged when the traffic_manager was
removed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] SolidWallOfCode commented on pull request #9212: traffic_ctl - Fix number of mandatory arguments when sending a plugin msg.
Posted by GitBox <gi...@apache.org>.
SolidWallOfCode commented on PR #9212:
URL: https://github.com/apache/trafficserver/pull/9212#issuecomment-1322114606
Actshually, if you're fiddling with that, requiring two arguments is annoying. See [here](http://docs.solidwallofcode.com/txn_box/install.en.html#reloading) for an example, where to get TxnBox to reload its configuration, you have to provide a second argument that is useless. If there's no second argument, then an empty (zero length) message should be sent.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] bneradt merged pull request #9212: traffic_ctl - Fix number of mandatory arguments when sending a plugin msg.
Posted by GitBox <gi...@apache.org>.
bneradt merged PR #9212:
URL: https://github.com/apache/trafficserver/pull/9212
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] brbzull0 commented on pull request #9212: traffic_ctl - Fix number of mandatory arguments when sending a plugin msg.
Posted by GitBox <gi...@apache.org>.
brbzull0 commented on PR #9212:
URL: https://github.com/apache/trafficserver/pull/9212#issuecomment-1322202567
> Actshually, if you're fiddling with that, requiring two arguments is annoying. See [here](http://docs.solidwallofcode.com/txn_box/install.en.html#reloading) for an example, where to get TxnBox to reload its configuration, you have to provide a second argument that is useless. If there's no second argument, then an empty (zero length) message should be sent.
yes, it supposed to be at least 1 arg, it was enforing 2. This fixes that.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org