Conversation
@@ -5,6 +5,9 @@ on [Keep a CHANGELOG](http://keepachangelog.com/). This project adheres to | |||
|
|||
## [Unreleased] | |||
|
|||
### Changed | |||
- Improved logging for annotation scanning |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you didn't add the change for this commit.
@@ -42,14 +42,15 @@ public void initialise() { | |||
* | |||
* @param envelope - the envelope to be audited. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
additional param for javadoc? - or just remove if inheriting from interface.
} | ||
|
||
@Test(expected = IllegalStateException.class) | ||
public void shouldComponentIsNotSetThrowException() throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldThrowExceptionIfComponentNotSet ?
@@ -19,16 +19,18 @@ | |||
|
|||
@Override | |||
public InterceptorContext process(final InterceptorContext interceptorContext, final InterceptorChain interceptorChain) { | |||
recordAudit(interceptorContext.inputEnvelope()); | |||
final String component = (String) interceptorContext.getInputParameter("component") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could we add getComponent() on InterceptorContext?
Presumably it is used elsewhere already so makes sense to have it in one place and also not expose the "component" value?
73e1422
to
529d4c5
Compare
This is a workaround to allow audit to know which component within a service a message is coming from, until we have a better way of discovering the context, service, and componet names.
529d4c5
to
ebb7f05
Compare
No description provided.