mace-opensaml-users - Re: Revision 921 ArtifactResponseImpl.java
Subject: OpenSAML user discussion
List archive
- From: Chad La Joie <>
- To:
- Subject: Re: Revision 921 ArtifactResponseImpl.java
- Date: Wed, 16 May 2007 17:15:42 -0500
- Organization: University Information Systems
Ah yep, fixed in 939
Jeff Brewer wrote:
> The following method was added in revision 921 (from May 2^nd ).
>
>
>
> /**
> {@inheritDoc}
> */
>
> *public* *void* setMessage(SAMLObject message) {
>
> prepareForAssignment(protocolMessage, message);
>
> }
>
>
>
> When I use getMessage() on an ArtifactResponse object, it always returns
> null because the protocolMessage member variable is never actually set.
> I think the method should be written:
>
> /**
> {@inheritDoc}
> */
>
> *public* *void* setMessage(SAMLObject message) {
>
> this. protocolMessage = prepareForAssignment(protocolMessage,
> message);
>
> }
>
>
>
> Please let me know what you think.
>
>
>
> Jeff
>
--
Chad La Joie 2052-C Harris Bldg
OIS-Middleware 202.687.0124
- Revision 921 ArtifactResponseImpl.java, Jeff Brewer, 05/16/2007
- Re: Revision 921 ArtifactResponseImpl.java, Chad La Joie, 05/16/2007
Archive powered by MHonArc 2.6.16.