You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@directory.apache.org by "Kai Zheng (JIRA)" <ji...@apache.org> on 2016/01/07 12:23:39 UTC

[jira] [Commented] (DIRKRB-523) Avoid outputting data to System.out in Asn1 + Asn1Dumper

    [ https://issues.apache.org/jira/browse/DIRKRB-523?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15087227#comment-15087227 ] 

Kai Zheng commented on DIRKRB-523:
----------------------------------

I thought the {{dump}} is clear in its meaning that outputs something in the console. It's very helpful in relevant troubleshooting but I guess some time when it's stable then we can let them shut up during the building. We just comment out the calling places in the tests. Let's do this in this issue.

Regarding in the library, the API itself please keep it. It's planned to allow some method to get the content back so the callers can decide where to output. 

> 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)