It's basically inconsequential, but in the essex backport for TokenController, there's a redundant three lines validating:
len(tenant_id) > CONF.max_param_size
Also, there's some trivial refactoring (pulling CONF into the local namespace) in the grizzly patch that you backported to folsom. +1 for the refactoring, but I'm not sure it should be included in the backport?
It's basically inconsequential, but in the essex backport for TokenController, there's a redundant three lines validating:
len(tenant_id) > CONF.max_param_size
Also, there's some trivial refactoring (pulling CONF into the local namespace) in the grizzly patch that you backported to folsom. +1 for the refactoring, but I'm not sure it should be included in the backport?