Skip to content

a little contribution#83

Open
zgq456 wants to merge 3 commits into
audit4j:masterfrom
zgq456:master
Open

a little contribution#83
zgq456 wants to merge 3 commits into
audit4j:masterfrom
zgq456:master

Conversation

@zgq456

@zgq456 zgq456 commented Sep 18, 2018

Copy link
Copy Markdown

I have done some optimization for audit4j after using this for some time, including

  1. print precise argument name instead of arg0, 1.... in conjunction with jdk8 feature
  2. deep size check to avoid infinite recursion (manual skip http request/response/session heavy objects, maybe you can expose configuration to skip user custom class , eg joda LocalDateTime, LocalDate )
  3. release resource to avoid too many open files in Linux , this is fatal, there was an existed issue opened raised by someone found in solaris

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant