[Seaside] Issue with moving from Seaside 2.6 to 2.8...

[Richard E. Flower]

Boris --

I tried this and I don't appear to have any message "ipAddressString"  
in my image.. Also,
what is the local "wsResponse" variable for -- I didn't see it  
referenced anywhere.  Thx!

Below is my current cut at things and it doesn't appear to be very  
happy:

doTheWorkFor: aTransport

	| waHandler waRequest wsResponse |
	waRequest := message.
	waRequest headers at: 'x-forwarded-for' put: aTransport id  
ipAddressString.
	waHandler := aTransport marshaler handlerForRequest: waRequest.
	^aTransport useStreaming
[ ... ]


-- Rick

On Jun 11, 2008, at 9:09 AM, Boris Popov wrote:

> Oh, yea, I remember now. I raised an issue with Cincom on the list,  
> but
> got no resolution back. The problem as I remember it is that there  
> is no
> way to get at the client IP from the native Opentalk request. Here's  
> the
> override I put in place here to get around the issue,
>
> SeasideRequest>>doTheWorkFor: aTransport
>
> | waHandler waRequest wsResponse |
> waRequest := message.
> waRequest headers at: 'x-forwarded-for' put: aTransport id
> ipAddressString.
> [ ... the rest of the method as-is ... ]
>
> I wouldn't mind a more proper fix from the Opentalk guys though,  
> this is
> a nasty hack.
>
> Cheers,
>
> -Boris
>
> -- 
> +1.604.689.0322
> DeepCove Labs Ltd.
> 4th floor 595 Howe Street
> Vancouver, Canada V6C 2T5
> http://tinyurl.com/r7uw4
>
> boris at deepcovelabs.com
>
> CONFIDENTIALITY NOTICE
>
> This email is intended only for the persons named in the message
> header. Unless otherwise indicated, it contains information that is
> private and confidential. If you have received it in error, please
> notify the sender and delete the entire message including any
> attachments.
>
> Thank you.
>
>> -----Original Message-----
>> From: seaside-bounces at lists.squeakfoundation.org [mailto:seaside-
>> bounces at lists.squeakfoundation.org] On Behalf Of Rick Flower
>> Sent: Wednesday, June 11, 2008 9:00 AM
>> To: Seaside - general discussion
>> Subject: Re: [Seaside] Issue with moving from Seaside 2.6 to 2.8...
>>
>> On Wed, June 11, 2008 7:18 am, Boris Popov wrote:
>>> Never mind, sorry, didn't scroll far enough. That looks like the
> problem
>>> that was fixed a while ago, I would load the latest from public
>> repository
>>> and check again.
>>
>> Boris --
>>
>> I picked the latest version out of the Cincom repository last night
> (rev
>> 541 if I recall) of the vanilla "Seaside" bundle.. Is the fix not in
> that
>> version? Thx!
>>
>>
>> _______________________________________________
>> seaside mailing list
>> seaside at lists.squeakfoundation.org
>> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>