Intscala 55 - logging channel adapter DSL components

[sobychacko]

No description provided.

[ghillert]

Hi @sobychacko,

Could you rebase this PR? It looks like this PR contains some more code for INTSCALA-54. Also, please add one or two tests to org.springframework.integration.dsl. Once that is done, I think I can merge the PR.

Thanks!

[sobychacko]

@ghillert PR is rebased and updated with unit test.

[ghillert]

I think we should call this log (like the vert to log ;-)

[ghillert]

I feel the class-name for the logger is not perfect, yet - Instead of LoggingChannelAdapterEndpoint.scala, maybe better call it LoggerEndpointDsl.scala. I believe that aligns better with the rest of the DSL.

[ghillert]

@sobychacko Sorry, did finally some more thinking about it...

I think we also should support the following optional properties:

• logFullMessage (true | false) (Default: false)
• logLevel
  • fatal
  • error
  • warn
  • info
  • debug
  • trace

The default log-level shall be info.

• Furthermore, we should support the logging of SpEL expressions.
• The logger name shall be optional
  • Internally (Spring Integration) - By default (If not specified) the logger name will be the fully qualified class name of the LoggingHandler implementation

Not sure, yet, what the perfect DSL would look like...

inbound.poll("...").withFixedDelay(100) --> log("org.springintegration.logger")
inbound.poll("...").withFixedDelay(100) --> log
inbound.poll("...").withFixedDelay(100) --> log("org.springintegration.logger").withErrorLevel().logFullMessage(true)
inbound.poll("...").withFixedDelay(100) --> log("org.springintegration.logger").withLogLevel(ERROR).logFullMessage(true)
inbound.poll("...").withFixedDelay(100) --> log().withErrorLevel().logFullMessage(true)

[sobychacko]

@ghillert PR is updated again by incorporating the comments you made. Please take a look and see what you think.