[jira] Updated: (WSS-20) Better error reporting from CryptoFactory.loadClass() [patch]

     [ http://issues.apache.org/jira/browse/WSS-20?page=all ]

Thomas Leonard updated WSS-20:
------------------------------

    Attachment: CryptoFactory.patch

> Better error reporting from CryptoFactory.loadClass() [patch]
> -------------------------------------------------------------
>
>          Key: WSS-20
>          URL: http://issues.apache.org/jira/browse/WSS-20
>      Project: WSS4J
>         Type: Improvement
>     Reporter: Thomas Leonard
>     Assignee: Davanum Srinivas
>  Attachments: CryptoFactory.patch
>
> loadClass() first tries to construct an instance of cryptogenClass with a 
> single properties parameter to the constructor. If this fails, it tries again 
> with no arguments. If this second attempt fails, it reports a generic message 
> saying "cannot create instance". This is not very helpful, as it does not 
> pass on any error reported by the code.
> Perhaps it would be better to check which constructor exists, use that, and 
> report any error back to the user? I have a patch to do this, which I will 
> attach.
> Note: This causes a change in behaviour if the class has *both* constructors, 
> since it will only try the one which takes the parameter. However, I think 
> this is more likely to be the correct behaviour.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira