[jira] [Commented] (TC-490) mso.qstring_handling parameter is checked but not documented

["Jeremy Mitchell (JIRA)" <ji...@apache.org>]

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

Jeremy Mitchell commented on TC-490:
------------------------------------

I thought but you'd have to ask [~dg4prez] we were moving away from these UI config file routes to the API config file routes - https://github.com/apache/incubator-trafficcontrol/blob/master/traffic_ops/app/lib/API/Configs/ApacheTrafficServer.pm

so be sure that if any changes are made in UI/ConfigFiles.pm, they are also made in ApacheTrafficServer.pm

> mso.qstring_handling parameter is checked but not documented
> ------------------------------------------------------------
>
>                 Key: TC-490
>                 URL: https://issues.apache.org/jira/browse/TC-490
>             Project: Traffic Control
>          Issue Type: Bug
>          Components: Traffic Ops API
>    Affects Versions: 2.1.0
>            Reporter: Matt Mills
>            Priority: Blocker
>
> mso.qstring_handling not listed in docs, seems to be defined elsewhere as psel.qstring_handling
>    but MID parent selection code checks if mso.qstring_handling is not defined
> {code}
>    my $qsh            = $ds->{'param'}->{'parent.config'}->{'mso.qstring_handling'};
>    my $parent_qstring = "ignore";                                                      # default is ignore, unless for alg consistent_hash
>    if ( !defined($qsh) && $mso_algorithm eq 'consistent_hash' && $ds->{qstring_ignore} == 0 ) {
>       $parent_qstring = 'consider';
>    }
> {code}
> The parameter is unlikely to ever exist, since it's not documented anywhere, but either way either the code should be corrected or the documentation should be.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)