Github user pmujumdar commented on the pull request:
https://github.com/apache/incubator-eagle/pull/43#issuecomment-170839212
@sunlibin Thanks for the feedback.
I see there are a bunch of regex checks already in that code path. But this
is called on every log entry, fair enough. I updated the patch to restore the
old code and added the missing case for non-kerberos principal.
Also since removed some of the trim() methods which are not really needed
which should help the performance a bit more.
Rebased the branch and ran tests successfully in parent directory
eagle-security. Fixed a couple of license header issues that hit on latest
master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
