-
-
Notifications
You must be signed in to change notification settings - Fork 3
Search or highlight by body contents. #10
Comments
there are several ways to solve this problem:
what do you think? do you have more ideas ? |
Thanks for the write-up @cle1000. My five cents: 1 is a possible resource hog - not what we want. 3 and 4 are nice solutions, but this implies keeping state on the server and interleaving filter/highlight logic with the whole flow list components. I'm a bit a fraid that this ends up as spaghetti. 2 is not really pretty, but leaves all flow content searching in an isolated place/API. The main issue here would be that once we receive a flow update, we need to do an asynchronous request to the server, leaving a small time period in between where we don't know how to handle the flow. for positive/"show only" filters that probably wouldn't be too bad, as they just stay hidden for a bit longer. For negative filters, we'd have flows show up incorrectly and then quickly disappear again - that sucks, but might be enough of an edge case that we don't need to care about it. |
@cortesi: what do you think? i should implement this in the next two weeks as my "last" gsoc feature :) |
any update on this feature? |
Filters for request/response body are currently not implemented. This is a tricky feature, as we do not have the bodies in memory on the client side and we kind of need to query the server for this.
The text was updated successfully, but these errors were encountered: