Skip to Content.
Sympa Menu

mace-opensaml-users - Re: Revision 921 ArtifactResponseImpl.java

Subject: OpenSAML user discussion

List archive

Re: Revision 921 ArtifactResponseImpl.java


Chronological Thread 
  • 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



Archive powered by MHonArc 2.6.16.

Top of Page