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 2020/09/02 17:16:22 UTC

[GitHub] [trafficserver] bneradt opened a new pull request #7159: MicroDNS Extension: handle different 'default' types

bneradt opened a new pull request #7159:
URL: https://github.com/apache/trafficserver/pull/7159


   The MicroDNS extension supports populating the 'otherwise' node with a
   list of hosts via the default parameter.  The Traffic Server tests only
   pass a string to default, but tests could also pass a list. This makes
   the extension deal with either of these input types. As it is, without
   this change, tests cannot pass a list because the list itself would be
   wrapped as a list.
       
   This change fixes the use of MicroDNS for plugins that use the
   Traffic Server extensions, such as txn_box.so.
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficserver] zwoop commented on pull request #7159: MicroDNS Extension: handle different 'default' types

Posted by GitBox <gi...@apache.org>.
zwoop commented on pull request #7159:
URL: https://github.com/apache/trafficserver/pull/7159#issuecomment-687219176


   Cherry-picked to v9.0.x branch.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficserver] bneradt commented on pull request #7159: MicroDNS Extension: handle different 'default' types

Posted by GitBox <gi...@apache.org>.
bneradt commented on pull request #7159:
URL: https://github.com/apache/trafficserver/pull/7159#issuecomment-685885149


   I've marked this for 9.0.x because it will have this problem as well after we merge the URL parse fix for empty paths to it:
   https://github.com/apache/trafficserver/pull/7119


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficserver] bneradt merged pull request #7159: MicroDNS Extension: handle different 'default' types

Posted by GitBox <gi...@apache.org>.
bneradt merged pull request #7159:
URL: https://github.com/apache/trafficserver/pull/7159


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org