Traffic Server
  1. Traffic Server
  2. TS-81

Have one single place to store and lookup remap rules irrespective of type

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.0.0a
    • Fix Version/s: 5.3.0
    • Component/s: Core
    • Labels:

      Description

      Currently, remap rules are stored in different structures and looked up separately based on type (forward, reverse, etc.). It'd be better design and more maintainable to process (store, search) all rules in one structure and then use type to determine action.

      A fundamental problem with our current implementation is that the order in remap.config is not honored. E.g. a map directive always takes precedence over any redirect directives (if both matches).

        Issue Links

          Activity

          Hide
          Leif Hedstrom added a comment -

          I'm going to target this for v2.1.x for now, maybe as an interim solution, we can use the "priority" numbering to decide if a map rule can be beaten by a lookup on the redirect maps (similar to how we optimize the regex mappings).

          Show
          Leif Hedstrom added a comment - I'm going to target this for v2.1.x for now, maybe as an interim solution, we can use the "priority" numbering to decide if a map rule can be beaten by a lookup on the redirect maps (similar to how we optimize the regex mappings).
          Hide
          Leif Hedstrom added a comment -

          From TS-2142 by igalic:

          There's a widely held believe in our community that redirects happen before map rules in remap.config. Or that the order in remap.config has more of an affect than the actual precedence.
          I'm not entirely sure which of those two we should cater to.
          Complete top-down evaluation might turn out to be just as constrictive as our current model, one way or the other, the current behaviour is dissatisfying.

          Show
          Leif Hedstrom added a comment - From TS-2142 by igalic: There's a widely held believe in our community that redirects happen before map rules in remap.config. Or that the order in remap.config has more of an affect than the actual precedence. I'm not entirely sure which of those two we should cater to. Complete top-down evaluation might turn out to be just as constrictive as our current model, one way or the other, the current behaviour is dissatisfying.
          Show
          Leif Hedstrom added a comment - Moving to 5.0.0 as per https://cwiki.apache.org/confluence/display/TS/New+Release+Processes

            People

            • Assignee:
              Unassigned
              Reporter:
              Manjesh Nilange
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:

                Development