-
Notifications
You must be signed in to change notification settings - Fork 210
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added UI panel for http header filters #1345
Conversation
@@ -20,7 +21,8 @@ | |||
FilterChainFactory filterChainFactory() { | |||
List<SubscriptionMessageFilterCompiler> subscriptionFilterCompilers = Arrays.asList( | |||
new AvroPathSubscriptionMessageFilterCompiler(), | |||
new JsonPathSubscriptionMessageFilterCompiler() | |||
new JsonPathSubscriptionMessageFilterCompiler(), | |||
new HeaderSubscriptionMessageFilterCompiler() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I remember, hermes-management is aware of the filters only because it has a logic related to the filters debugger. As the debugger doesn't currently support header filters, I think this change is not needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right, this change is not needed. However I added it to simplify adding support for headers dubbuger in the future. Thanks to this line only changes on frontend side are needed now and next developer doesn't have to analyze backend side at all.
@@ -29,8 +29,9 @@ console: | |||
label: DAYS | |||
subscription: | |||
showHeadersFilter: true | |||
showFixedHeaders: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should also be added to ConsoleProperties.
Added
Http header filter
panel in subscription view. The panel is displayed belowFilter
panel as in the attached image. By defaultHttp header panel
is disabled. It can be enabled byshowHeadersFilter
flag in Hermes console config. At the moment the panel dosen't supportDebug
mode andmatching strategy
, but they can be added in the future.