OpenSAML user discussion

["Cortezzo, Jason" <>]

May I recommend changing this constructor of org.opensaml.SAMLException
from....

    public SAMLException(String msg, Exception e) {
        super(msg);
        this.msg = msg;
        this.e = e;
    }

...to...

    public SAMLException(String msg, Exception e) {
        super(msg, e);  // Nest the exceptions
        this.msg = msg;
        this.e = e;
    }


It will take advantage of the "chained exception facility".  This way if you
do a stacktrace on the SAMLException, you get the stacktrace of the cause as
well.


-Jason Cortezzo