Results 224 comments of Alexander Yastrebov

@szuecs > So it's not really problem but a general one that we might want to fix There are three things to consider `request.Host` used by `Host` predicate, `Host` header...

Probably could be hacked around with `JWTPayloadAnyKVRegexp` predicate and specific rate-limited routes

I mean that by having specific route with `JWTPayloadAnyKVRegexp` users can isolate particular client(+scopes) and then impose `clusterRatelimit` on it without relying on `Authorization` header (though it must be validated).

> Any thoughts, or other alternatives? The Path spec https://github.com/zalando/skipper/blob/master/docs/reference/predicates.md#path says > The wildcards must follow a / So maybe we may lift-up the `:*` check somehow https://github.com/zalando/skipper/blob/f27cb2d71530acf167e5c06e721a9b0286dbc9cb/pathmux/tree.go#L119-L121 and reduce...

While exploring the filter implementations I've found filters that support duration string argument and number argument interpreted as number of _milliseconds_ (e.g. `consecutiveBreaker`). There are also filters that support duration...

The converter could be useful for predicates as well so maybe it should be moved to `eskip` or neighbor package as it defines both `Filter` and `Predicate` interface that use...

I like the idea in general. Should we consider supporting [label selector syntax](https://github.com/kubernetes/apimachinery/blob/31bc292891b6a196ad41946423c71bbe39c05d3c/pkg/labels/selector.go#L846-L884) instead of simplistic name-value set?

I think we need to better understand how well or not the pool actually behaves before making it more complex. E.g. I have added a log statement to see when...

> What I do not understand though is that with 100 rps and 1s latency there should be 100 requests in parallel which with the default state pool size of...