Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/
This has been merged into the feature branch. Can you close
@simonellistonball?
---
Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/
Thanks @simonellistonball. I think this is good enough to start with. +1
---
Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/
Here is what I think is pending on this PR:
- Address the minor comments I've made inline
- Update the metron-rest and/or other relevant READMEs with the new
authentication approach
-
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/
Thanks for the catches Ryan.
My objective around metron-rest was to try and leave it as little changed
as possible in this effort, since the ticket is more around the ui hosts,
Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/
I really like this feature and think it's an important improvement. This
PR is a great start and I don't see any major architectural issues. I've added
some comments inline for the really minor