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. From: [hidden email] To: [hidden email] Sent: Wed Jun 11 07:12:29 2008 Subject: Re: [Seaside] Issue with moving from Seaside 2.6 to 2.8...
Who is calling it? A bit of stack print would help. I suspect its a session protector? It was probably fixed after release, did you load from public repository or parcels? From: [hidden email] To: Seaside - general discussion Sent: Wed Jun 11 07:09:43 2008 Subject: [Seaside] Issue with moving from Seaside 2.6 to 2.8... Hi all.. I'm back to work some more Seaside code after having been away from it for almost a year. Anyway, I've got a new VW 7.6 image loaded up with the latest Seaside/Glorp + others and then I loaded my app and tried running it and get the following stack dump when the code is looking for #remoteAddress on a Seaside.WARequest object.. Any ideas on what's messed up? I did notice that the config page of Seaside is vastly different when configuring each app and I'm impressed with how that works now -- even my Glorp stuff is showing up there -- very slick! Anyway, if you've got any ideas on this issue below, that would be great! MTIA! -- Rick Message not understood: #remoteAddressPossible Causes
Stack Trace
_______________________________________________ seaside mailing list [hidden email] http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside |
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 [hidden email] http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside |
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 [hidden email] 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: [hidden email] [mailto:seaside- > [hidden email]] 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 > [hidden email] > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside _______________________________________________ seaside mailing list [hidden email] http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside |
Thanks Boris.. I'll give it a shot tonight!
On Wed, June 11, 2008 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 > > [hidden email] > > 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: [hidden email] [mailto:seaside- >> [hidden email]] 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 >> [hidden email] >> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside > > _______________________________________________ seaside mailing list [hidden email] http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside |
In reply to this post by Boris Popov, DeepCove Labs (SNN)
Yup, sorry about the delay. Created AR#54545: "WASessionProtector needs the SeasideRequest to provide #remoteAddress"
for this. Should have an "official" fix shortly. Martin 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 > _______________________________________________ seaside mailing list [hidden email] http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside |
In reply to this post by Boris Popov, DeepCove Labs (SNN)
Thank you. I'm not sure if my fix is really a fix, but it was the least intrusive override I could find. _______________________________________________ seaside mailing list [hidden email] http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside |
In reply to this post by Boris Popov, DeepCove Labs (SNN)
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 > > [hidden email] > > 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: [hidden email] [mailto:seaside- >> [hidden email]] 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 >> [hidden email] >> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside > _______________________________________________ seaside mailing list [hidden email] http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside |
Free forum by Nabble | Edit this page |