merrimanr commented on issue #1275: METRON-1878: Add Metron as a Knox service URL: https://github.com/apache/metron/pull/1275#issuecomment-449399542 1. Can you talk about the rewrite rules at all? What are they currently doing and when would someone need to modify those files? There is not much to say about this because there isn't any significant rewriting going on. Knox is simply forwarding the requests as is. This is the same for all 3 services (Alerts UI, Management UI, and REST) and you'll notice all `rewrite.xml` files are structured the same. The only requirement I had an issue with is that static assets needs to be prepended with the Knox path. This was solved by making the base href relative in `index.html`. I added a comment in the metron-alerts `index.html` file but missed it in metron-config. I will go back and add that comment. I cannot think of a scenario where changing a `rewrite.xml` file would be necessary. 2. Can you add any comment on the LDAP config with roles? Maybe a very brief cookie cutter example of how a user's LDAP users and groups will translate into Metron's interpretation of that info. You're right, we should document how this works. I will add a section to the README.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
