[
https://issues.apache.org/jira/browse/DIRKRB-523?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15087262#comment-15087262
]
Kai Zheng commented on DIRKRB-523:
----------------------------------
In my understanding, {{log}} is to log something that's seldom the formal
output in a process. It's just in case you need it. However, {{dump}} is an
intended call that's wanted to be out right before his hand. If you'd like to
check out other similar dumper tools, they often dump to the user console. It's
simple, if you don't want it, then don't call it.
It's different from something like in Kerby KdcServer side, we need to log the
message out to the server logs, where the dumper will be equipped with a new
method to get the content out and then the content can be output to logs when
meets with some errors for troubleshooting.
> Avoid outputting data to System.out in Asn1 + Asn1Dumper
> --------------------------------------------------------
>
> Key: DIRKRB-523
> URL: https://issues.apache.org/jira/browse/DIRKRB-523
> Project: Directory Kerberos
> Issue Type: Bug
> Reporter: Colm O hEigeartaigh
>
> I notice a lot of text output in the Kerby tests along the lines of:
> Dumping data:
> >60818561101A044A6F686E1A01501A05536D697468A00A1A084469726563746F72420133A10A43083139373130393137A212
> 61101A044D6172791A01541A05536D697468A342311F61111A0552616C70681A01541A05536D697468A00A43083139353731
> 313131311F61111A05537573616E1A01421A054A6F6E6573A00A43083139353930373137
> application [0] [tag=0x60, off=0, len=3+133]
> etc.
>
> Both the Asn1 + Asn1Dumper classes are outputting data using
> System.out.println(). I don't think this is a good idea, that classes in
> src/main/java are outputting data to system.out. Is there a reason for
> this? Surely if anything, the Asn1Dumper could output to a log instead.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
