Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Let LogOperationAspect report parameters that are not basic types #62

Closed
Verdoso opened this issue May 4, 2020 · 1 comment · Fixed by #63
Closed

Let LogOperationAspect report parameters that are not basic types #62

Verdoso opened this issue May 4, 2020 · 1 comment · Fixed by #63

Comments

@Verdoso
Copy link
Owner

Verdoso commented May 4, 2020

Right now, LogOperationAspect just reports parameteres that are primitives, enums, based on CharSequence or a Collection of the those.
It would be nice if we allowed other types to be logged, even though the responsability of the String represetation should fall on the developer.

Initial idea would be to add an Interface with a String returning method that could be used to signal that that parameter can be looged and provide the method to obtain the represeration.

Null parameters will still be ignored

@Verdoso
Copy link
Owner Author

Verdoso commented May 4, 2020

The interface Logable has been added. A parameter that implements this interface will also be logged

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 a pull request may close this issue.

1 participant