WA Authentication

> When I do:
>   h := (WADispatcher default handlerAt: 'status').
>   h configuration addParent: WAAuthConfiguration instance.
>   h preferenceAt: #login put: 'admin'.
>   h preferenceAt: #passwordHash put: (GRPlatform current secureHashFor:
> '123').
>   h addFilter: WAAuthenticationFilter new.
> per [1] and [2], I get a 404 on a ZnRequest(GET /favicon.ico). This is
> possibly similar to [3], but there didn't seem to be a resolution...
>
> I tried this with a few of the built-in handlers, all with the same result.
> How do I fix this?

> Philippe Marschall wrote
>> Do you have a hander registered that should handle /favicon.ico?
>
> It's a vanilla Seaside loaded via the mailing list script into Pharo 2.0. So
> I guess not out of the box, but I saved the bytes as a file, so I thought
> maybe the static file server would serve it (which it did, I got an OK
> response for that file, but that didn't help the situation with my
> component)
>
>
> Philippe Marschall wrote
>>  If you don't add the
>> WAAuthenticationFilter does it work?
>
> The page loads as if there was no authentication
>
>
> Philippe Marschall wrote
>> Hard to say without debugging.
>
> It's just a test image. I'd be happy to send it to you if you'd be willing
> to take a look...
>
> Thanks,
> Sean
>
>
>
> --
> View this message in context: http://forum.world.st/WA-Authentication-tp4669095p4669341.html
> Sent from the Seaside General mailing list archive at Nabble.com.
> _______________________________________________
> seaside mailing list
> [hidden email]
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>

> Doing that in Seaside 3.0.7/Pharo 1.4 gives two deprecation warnings.
>
>
> "Deprecation: The method SmalltalkImage>>isRunningCog has been deprecated.
> Use Smalltalk vm isRunningCog instead"
>
>
> and
>
>
> "Deprecation: The method SmalltalkImage>>getSystemAttribute: has been
> deprecated.
> This is a private method of Virtual machine. Use Smalltalk vm
> <appropriateName> for retreiving attribute(s)
> "
>
>
>
> after deleting the deprecation warnings (or changing the code to use the
> warnings suggested replacement) it is able to authenticate.
>
>
>
>
>
>
>
> On 02/11/2013 07:23 PM, Sean P. DeNigris wrote:
>> Philippe Marschall wrote
>>> Do you have a hander registered that should handle /favicon.ico?
>>
>> It's a vanilla Seaside loaded via the mailing list script into Pharo 2.0. So
>> I guess not out of the box, but I saved the bytes as a file, so I thought
>> maybe the static file server would serve it (which it did, I got an OK
>> response for that file, but that didn't help the situation with my
>> component)
>>
>>
>> Philippe Marschall wrote
>>>  If you don't add the
>>> WAAuthenticationFilter does it work?
>>
>> The page loads as if there was no authentication
>>
>>
>> Philippe Marschall wrote
>>> Hard to say without debugging.
>>
>> It's just a test image. I'd be happy to send it to you if you'd be willing
>> to take a look...
>>
>> Thanks,
>> Sean
>>
>>
>>
>> --
>> View this message in context: http://forum.world.st/WA-Authentication-tp4669095p4669341.html
>> Sent from the Seaside General mailing list archive at Nabble.com.
>> _______________________________________________
>> seaside mailing list
>> [hidden email]
>> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>>
>

> Philippe Marschall wrote
>> Do you have a hander registered that should handle /favicon.ico?
>
> It's a vanilla Seaside loaded via the mailing list script into Pharo 2.0. So
> I guess not out of the box, but I saved the bytes as a file, so I thought
> maybe the static file server would serve it (which it did, I got an OK
> response for that file, but that didn't help the situation with my
> component)
>
>
> Philippe Marschall wrote
>>  If you don't add the
>> WAAuthenticationFilter does it work?
>
> The page loads as if there was no authentication

> I may have found the problem...
>
> In Pharo 2.0, ZnSeasideStaticServerAdaptorDelegate>>handleRequest: adds a
> few lines:
>     response isError
> ifFalse: [ ^ response ].
> ^ self staticDelegate handleRequest: znRequest
>
> changing to:
>     (response isError and: [  response isAuthenticationRequired not ])
> ifFalse: [ ^ response ].
> ^ self staticDelegate handleRequest: znRequest
>
> makes everything work!
>
> I'll check with Sven...
>
> Thanks for the help :)
> - Sean
>
>
>
> --
> View this message in context: http://forum.world.st/WA-Authentication-tp4669095p4669480.html
> Sent from the Seaside General mailing list archive at Nabble.com.
> _______________________________________________
> seaside mailing list
> [hidden email]
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>

> I may have found the problem...
>
> In Pharo 2.0, ZnSeasideStaticServerAdaptorDelegate>>handleRequest: adds a
> few lines:
>    response isError
> ifFalse: [ ^ response ].
> ^ self staticDelegate handleRequest: znRequest
>
> changing to:
>    (response isError and: [  response isAuthenticationRequired not ])
> ifFalse: [ ^ response ].
> ^ self staticDelegate handleRequest: znRequest
>
> makes everything work!
>
> I'll check with Sven...
>
> Thanks for the help :)
> - Sean

> On Feb 12, 2013, at 2:52 PM, Sven Van Caekenberghe-2 [via Smalltalk] <[hidden email]> wrote:
> > Yes the fix looks good to me, it can be integrated in the Seaside adaptor, who will do it ?
>
> Here's a MC package. I'm happy to upload it if you add me to the repo... lmk...
>
>
>        
>
>  Zinc-Seaside-SeanDeNigris.39.mcz (19K) Download Attachment
>
> View this message in context: Re: WA Authentication
> Sent from the Seaside General mailing list archive at Nabble.com.
> _______________________________________________
> seaside mailing list
> [hidden email]
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside

> Sven Van Caekenberghe-2 wrote
>> Thanks for the patch, I uploaded it
>
> Great! Thanks for acting so quickly
>
>
> Sven Van Caekenberghe-2 wrote
>> the squeak source repository is open to all
>
> Okay, good to know
>
>
> Sven Van Caekenberghe-2 wrote
>> Did you mention in your original bug report that you were using
>> ZnSeasideStaticServerAdaptorDelegate ?
>
> No, I didn't realize it was a Zinc problem until I stepped through the code
> in both 1.4 and 2.0 and saw the difference. I used ZnZincStaticServerAdaptor
> per your load script at
> http://forum.world.st/Seaside-3-1-on-Pharo-2-0-td4645550.html