[TC-284] Strip "actionable" query strings

[limited]

If an "actionable" query string is present in the incoming URL, strip it to prevent duplication of object storage in case the delivery service's setting is to consider the qstring at the edge. If it is, leaving the qstring in place will potentially cause the same object to be stored twice; one object with the "actionable" TR query parameters and one without, if there are clients that make both types of requests.

Specific query strings are ?format=json, ?trred=false, and ?fakeClientIpAddress=foo.

Author: Jeff Elsloo
JIRA Link: https://issues.apache.org/jira/browse/TC-284
Found Version: 2.0.0

[rawlinp]

In general I think this would be good, but my worry is that there might be come clients/origins out there actually using a format query parameter. trred and fakeClientIpAddress seem less likely to be used by clients/origins since those are pretty TR-specific.

[ocket8888]

#3554 would add an HTTP header for format so that you can do that instead of using the query string parameter, to help avoid that collision

[rawlinp]

Indeed, if only we could get all our clients to start passing a new header instead of the old query parameter.