[rfc] [icedtea-web] providing little bit more debug outputs for few methods

Jiri Vanek jvanek at redhat.com
Mon Apr 30 05:13:15 PDT 2012


On 04/27/2012 05:29 PM, Deepak Bhole wrote:
>
Thank you for review!
> This patch is increasing verbosity and changing behavior (throwing a
The verbosity is not increased so much:)
-one message in (3) when SecurityDesc is null
-one message in debug mode enriched  for one path (1)
-one stacktrace printed out in debug mode (is printed out (without debug) in  in-browser-applet already)

If you do not like any of this changesets just say, I do not insists on any of them. Thy just come 
to me handy (and in case of consumed exception correct)
> different type of exception in getPermissions.

arrrgggh. Thanx for catch. Fixed. Exception is  printed and truly re-thrown (not copy of it at it 
was:-/) now.
>
> IMO it should be split..
As you wish :)
>
Changelogs:
debuggingEnchancements2_1_pathToKeystores.diff

2012-04-30 Jiri Vanek  <jvanek at redhat.com>

	* netx/net/sourceforge/jnlp/security/KeyStores.java: (getPathToKeystore):
	new method, able to search for file used for creating of KeyStore if possible
	* netx/net/sourceforge/jnlp/security/CertificateUtils.java: (inKeyStores)
	using getPathToKeystore for debug output.

debuggingEnchancements2_2_consumedExceptionPrintOut.diff

2012-04-30  Jiri Vanek  <jvanek at redhat.com>

	* netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java: (getPermissions):
	Any exception from this method is consumed somewhere. I have cough exception,
	reprint it in debug mode and re-throw (to be lost). Main condition in this
	method had several possible NullPointer exceptions. Separated and thrown before
	this condition.
	

debuggingEnchancements2_3_printOutMissingSecurityDesc.diff

2012-04-30  Jiri Vanek  <jvanek at redhat.com>

	Added more debuging outputs
	* netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java:
	(getCodeSourceSecurity): added output message when no SecurityDesc is found
	for some url/resource
	

Best regards, J.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: debuggingEnchancements2_3_printOutMissingSecurityDesc.diff
Type: text/x-patch
Size: 733 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20120430/dfad8229/debuggingEnchancements2_3_printOutMissingSecurityDesc.diff 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: debuggingEnchancements2_2_consumedExceptionPrintOut.diff
Type: text/x-patch
Size: 1899 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20120430/dfad8229/debuggingEnchancements2_2_consumedExceptionPrintOut.diff 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: debuggingEnchancements2_1_pathToKeystores.diff
Type: text/x-patch
Size: 2431 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20120430/dfad8229/debuggingEnchancements2_1_pathToKeystores.diff 


More information about the distro-pkg-dev mailing list