Github user cestella commented on the issue:
https://github.com/apache/metron/pull/771
+1 by inspection, great job @merrimanr
---
Github user iraghumitra commented on the issue:
https://github.com/apache/metron/pull/771
+1 works as described ...
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/771
definitely
---
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/771
@ottobackwards should we push that as a follow up issue for now rather than
expanding the scope of this PR?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/771
This brings up the issue with having 'split' these uis. They have common
things that we shouldn't have to duplicate in each application.
---
Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/771
The latest commit adds a "Session Expired" message on the login page and
also applies the fix to the Management UI. You will notice the 2 files look
identical between metron-alerts and
Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/771
That's not a bad idea. It occurs to me that this issue may also exist with
the Management UI and should be fixed there as well. Stay tuned.
---
Github user iraghumitra commented on the issue:
https://github.com/apache/metron/pull/771
Tested, the UI redirects to log-in after the timeout. Should we show a
message on the login page that session timeout happened ?.
---