Send modauthtkt-users mailing list submissions to
        modauthtkt-users@lists.sourceforge.net

To subscribe or unsubscribe via the World Wide Web, visit
        https://lists.sourceforge.net/lists/listinfo/modauthtkt-users
or, via email, send a message with subject or body 'help' to
        modauthtkt-users-requ...@lists.sourceforge.net

You can reach the person managing the list at
        modauthtkt-users-ow...@lists.sourceforge.net

When replying, please edit your Subject line so it is more specific
than "Re: Contents of modauthtkt-users digest..."


Today's Topics:

   1. Re: Query arg separator in generated URLs (Peter Karman)
   2. Re: ipv6 hashing support patch (Gavin Carr)
   3. Re: Query arg separator in generated URLs (Gavin Carr)
   4. Re: Query arg separator in generated URLs (Michael Peters)
   5. Can't run test suite (Michael Peters)
   6. Re: Can't run test suite (Michael Peters)
   7. Re: Query arg separator in generated URLs (Gavin Carr)


----------------------------------------------------------------------

Message: 1
Date: Wed, 10 Aug 2011 20:54:11 -0500
From: Peter Karman <pe...@peknet.com>
Subject: Re: [modauthtkt-users] Query arg separator in generated URLs
To: Michael Peters <mpet...@plusthree.com>,     mod_auth_tkt list
        <modauthtkt-users@lists.sourceforge.net>
Message-ID: <4e433643.6080...@peknet.com>
Content-Type: text/plain; charset=ISO-8859-1

Michael Peters wrote on 8/9/11 9:51 AM:

> Would there be any interest in having an extra configuration directive 
> that mod_auth_tkt could use to specify the separator, falling back to 
> the semi-colon if not specified? TktAuthQuerySeparator?
> 

+1 to that idea.


-- 
Peter Karman  .  http://peknet.com/  .  pe...@peknet.com



------------------------------

Message: 2
Date: Thu, 11 Aug 2011 17:15:57 +1000
From: Gavin Carr <ga...@openfusion.com.au>
Subject: Re: [modauthtkt-users] ipv6 hashing support patch
To: Chad Lavoie <lavo...@gmail.com>
Cc: modauthtkt-users@lists.sourceforge.net
Message-ID: <20110811071556.ga32...@openfusion.com.au>
Content-Type: text/plain; charset=us-ascii; format=flowed

Hi Chad,

On Sun, Jul 24, 2011 at 07:59:18PM -0400, Chad Lavoie wrote:
>-----BEGIN PGP SIGNED MESSAGE-----
>Hash: SHA1
>
>Hello,
>
>I have completed writing the patch I mentioned earlier for ipv6 ip
>hashing support, that will correctly hash the IP rather then cause a
>segfault.  I have applied this patch on my box, and my tests with ipv4
>and ipv6 have indicated that it does what its supposed to do.
>
>I am not sure that the way I have done it is correct(tm), and I am open
>to any suggestions/comments/insults.

Thanks a lot for the patch, and for raising this issue. Sorry for the
slow response.

Some initial comments:

- we should definitely fix the segfault with ipv6. I'd love a patch to
   just treat an ipv6 address as 0.0.0.0 initially (perhaps with a
   warning in the log?), while we figure out what else to do.

- I'm not sure how to handle ipv6 properly. However we do it we're going
   to be changing our ticket format, I guess, which will break existing
   client/library code. So will require a version bump, and some careful
   review to make sure we don't introduce any security issues.

- if we are going to change the ticket format, we should probably fix
   the timestamp as well, since that's only 32-bits as well, and is
   therefore prone to the Year 2038 overflow problem.

- is there a normalisation problem with ipv6? My understanding is ipv6
   ips can be shortened in various ways, all of which represent the same
   ip. Can we rely on the webserver giving us a normalised version, or do
   we need to handle that explicitly ourselves?

This is more a braindump than specific questions for you Chad. ;-)

FWIW, I've been moving to git along with the rest of the world. My
mod_auth_tkt tree is now here:

https://github.com/gavincarr/mod_auth_tkt

Cheers,
Gavin




------------------------------

Message: 3
Date: Thu, 11 Aug 2011 17:18:40 +1000
From: Gavin Carr <ga...@openfusion.com.au>
Subject: Re: [modauthtkt-users] Query arg separator in generated URLs
To: Peter Karman <pe...@peknet.com>
Cc: mod_auth_tkt list <modauthtkt-users@lists.sourceforge.net>,
        Michael Peters <mpet...@plusthree.com>
Message-ID: <20110811071840.gb32...@openfusion.com.au>
Content-Type: text/plain; charset=us-ascii; format=flowed

On Wed, Aug 10, 2011 at 08:54:11PM -0500, Peter Karman wrote:
>Michael Peters wrote on 8/9/11 9:51 AM:
>
>> Would there be any interest in having an extra configuration directive
>> that mod_auth_tkt could use to specify the separator, falling back to
>> the semi-colon if not specified? TktAuthQuerySeparator?
>>
>
>+1 to that idea.

Sure, I guess, if it's causing issues. Though I don't quite see where
Facebook is fitting into your request flow Michael? Can you elaborate?

FWIW, I've moved the code to github, for your patching pleasure:

   https://github.com/gavincarr/mod_auth_tkt

Cheers,
Gavin




------------------------------

Message: 4
Date: Thu, 11 Aug 2011 10:03:39 -0400
From: Michael Peters <mpet...@plusthree.com>
Subject: Re: [modauthtkt-users] Query arg separator in generated URLs
To: Peter Karman <pe...@peknet.com>,    mod_auth_tkt list
        <modauthtkt-users@lists.sourceforge.net>
Message-ID: <4e43e13b.30...@plusthree.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed

On 08/11/2011 03:18 AM, Gavin Carr wrote:

> Sure, I guess, if it's causing issues. Though I don't quite see where
> Facebook is fitting into your request flow Michael? Can you elaborate?

It's not intentionally fitting into our request flow :) We run a lot of 
different sites and our clients can set up protected portions of their 
sites and behind the scenes our code generates mod_auth_tktk .htaccess 
files, etc. Some of our client's members sometimes shared protected 
articles on facebook, etc with other members. Facebook doesn't just let 
people click through links, they are passed through their own tracking 
system and their system is making of mess of links with query strings 
that have semi-colons.

So even if the other person who sees it on facebook is a member they 
can't see the content because it gets mangled by facebook.

Facebook is just the latest example for us of a third party site getting 
those kinds of query strings wrong. And they're probably the biggest 
which is why we've been able to ignore it until now. But some of our 
clients are starting to complain even if it is facebook's fault.

-- 
Michael Peters
Plus Three, LP



------------------------------

Message: 5
Date: Thu, 11 Aug 2011 16:59:48 -0400
From: Michael Peters <mpet...@plusthree.com>
Subject: [modauthtkt-users] Can't run test suite
To: mod_auth_tkt list <modauthtkt-users@lists.sourceforge.net>
Message-ID: <4e4442c4.4000...@plusthree.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed

I'm trying to make sure I can run the test suite before doing some other 
development on mod_auth_tkt and I can't seem to get it to work.

First I tried:

   ./configure --apxs=/path/to/apxs
   make
   make test

And that fails with this error:
sed: can't read 
/home/mpeters/tmp/mod_auth_tkt/t/modules/mod_auth_tkt.la: No such file 
or directory
sed: can't read 
/home/mpeters/tmp/mod_auth_tkt/t/modules/mod_auth_tkt.la: No such file 
or directory
Warning!  dlname not found in 
/home/mpeters/tmp/mod_auth_tkt/t/modules/mod_auth_tkt.la.
Assuming installing a .so rather than a libtool archive.
chmod 755 /home/mpeters/tmp/mod_auth_tkt/t/modules/mod_auth_tkt.so
chmod: cannot access 
`/home/mpeters/tmp/mod_auth_tkt/t/modules/mod_auth_tkt.so': No such file 
or directory
apxs:Error: Command failed with rc=65536

So I did a:

   mkdir t/modules
   make test

And then using my test_shebang_env branch on github 
(https://github.com/mpeters/mod_auth_tkt/tree/test_shebang_env) I get 
test failures:
[warning] Using random number seed: 1295402170 (autogenerated)
t/40_htaccess.t ................ ok
t/10_cookie_expiry.t ........... 1/28 # Failed test 3 in 
t/10_cookie_expiry.t at line 37
# Failed test 6 in t/10_cookie_expiry.t at line 52
# Failed test 9 in t/10_cookie_expiry.t at line 66
# Failed test 12 in t/10_cookie_expiry.t at line 80
# Failed test 15 in t/10_cookie_expiry.t at line 94
# Failed test 18 in t/10_cookie_expiry.t at line 108
# Failed test 21 in t/10_cookie_expiry.t at line 122
# Failed test 24 in t/10_cookie_expiry.t at line 136
# Failed test 27 in t/10_cookie_expiry.t at line 150
t/10_cookie_expiry.t ........... Failed 9/28 subtests
t/22_timeout_guest_fallback.t .. skipped: env variable MAT_TEST_TIMEOUTS 
not set
t/31_vhost_global_secret.t ..... ok
t/21_timeout_refresh.t ......... skipped: env variable MAT_TEST_TIMEOUTS 
not set
t/07_guest_login_nocookie.t .... 1/11 # Failed test 3 in 
t/07_guest_login_nocookie.t at line 46
# Failed test 5 in t/07_guest_login_nocookie.t at line 52
# Failed test 8 in t/07_guest_login_nocookie.t at line 64
# Failed test 10 in t/07_guest_login_nocookie.t at line 70
t/07_guest_login_nocookie.t .... Failed 4/11 subtests
t/05_tokens.t .................. ok
t/12_cookie_secure.t ........... 1/8 # Failed test 3 in 
t/12_cookie_secure.t at line 37
# Failed test 6 in t/12_cookie_secure.t at line 49
t/12_cookie_secure.t ........... Failed 2/8 subtests
t/20_timeout.t ................. skipped: env variable MAT_TEST_TIMEOUTS 
not set
t/01_relative.t ................ ok
t/00_public.t .................. ok
t/02_bad.t ..................... ok
t/30_vhost_local_secret.t ...... ok
t/08_guest_user.t .............. 1/22 # Failed test 3 in 
t/08_guest_user.t at line 47
# Failed test 5 in t/08_guest_user.t at line 53
# Failed test 8 in t/08_guest_user.t at line 73
# Failed test 10 in t/08_guest_user.t at line 83
# Failed test 12 in t/08_guest_user.t at line 92
# Failed test 14 in t/08_guest_user.t at line 103
# Failed test 16 in t/08_guest_user.t at line 113
# Failed test 18 in t/08_guest_user.t at line 124
# Failed test 20 in t/08_guest_user.t at line 134
# Failed test 22 in t/08_guest_user.t at line 144
t/08_guest_user.t .............. Failed 10/22 subtests
t/15_secret_old.t .............. ok
t/50_bug_cookie_name.t ......... ok
t/07_guest_login.t ............. 1/6 # Failed test 3 in 
t/07_guest_login.t at line 46
# Failed test 5 in t/07_guest_login.t at line 52
t/07_guest_login.t ............. Failed 2/6 subtests
t/03_ignore_ip.t ............... ok
t/01_basic.t ................... ok
t/09_guest_not_allowed.t ....... 1/5 # Failed test 5 in 
t/09_guest_not_allowed.t at line 50
t/09_guest_not_allowed.t ....... Failed 1/5 subtests

Test Summary Report
-------------------
t/10_cookie_expiry.t         (Wstat: 0 Tests: 28 Failed: 9)
   Failed tests:  3, 6, 9, 12, 15, 18, 21, 24, 27
t/07_guest_login_nocookie.t  (Wstat: 0 Tests: 11 Failed: 4)
   Failed tests:  3, 5, 8, 10
t/12_cookie_secure.t         (Wstat: 0 Tests: 8 Failed: 2)
   Failed tests:  3, 6
t/08_guest_user.t            (Wstat: 0 Tests: 22 Failed: 10)
   Failed tests:  3, 5, 8, 10, 12, 14, 16, 18, 20, 22
t/07_guest_login.t           (Wstat: 0 Tests: 6 Failed: 2)
   Failed tests:  3, 5
t/09_guest_not_allowed.t     (Wstat: 0 Tests: 5 Failed: 1)
   Failed test:  5
Files=20, Tests=162,  7 wallclock secs ( 0.11 usr  0.04 sys +  4.96 cusr 
  0.83 csys =  5.94 CPU)
Result: FAIL
Failed 6/20 test programs. 28/162 subtests failed.
[warning] server localhost.localdomain:8529 shutdown
[  error] error running tests (please examine t/logs/error_log)
make[1]: *** [test_md5] Error 1
make[1]: Leaving directory `/home/mpeters/development/mod_auth_tkt/t'
make: *** [test] Error 2


I'm not sure how to run the tests to get more output (I tried what I 
thought was standard "make test TEST_VERBOSE=1" but that didn't work). 
Any help would be appreciated.

Thanks,

-- 
Michael Peters
Plus Three, LP



------------------------------

Message: 6
Date: Thu, 11 Aug 2011 17:02:41 -0400
From: Michael Peters <mpet...@plusthree.com>
Subject: Re: [modauthtkt-users] Can't run test suite
To: mod_auth_tkt list <modauthtkt-users@lists.sourceforge.net>
Message-ID: <4e444371.9080...@plusthree.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed

Funny enough, when I ran it again it did pass:

[warning] Using random number seed: 1215909723 (autogenerated)
t/30_vhost_local_secret.t ...... ok
t/07_guest_login.t ............. ok
t/05_tokens.t .................. ok
t/21_timeout_refresh.t ......... skipped: env variable MAT_TEST_TIMEOUTS 
not set
t/50_bug_cookie_name.t ......... ok
t/02_bad.t ..................... ok
t/07_guest_login_nocookie.t .... ok
t/12_cookie_secure.t ........... ok
t/15_secret_old.t .............. ok
t/03_ignore_ip.t ............... ok
t/22_timeout_guest_fallback.t .. skipped: env variable MAT_TEST_TIMEOUTS 
not set
t/01_relative.t ................ ok
t/00_public.t .................. ok
t/20_timeout.t ................. skipped: env variable MAT_TEST_TIMEOUTS 
not set
t/31_vhost_global_secret.t ..... ok
t/09_guest_not_allowed.t ....... ok
t/10_cookie_expiry.t ........... ok
t/08_guest_user.t .............. ok
t/40_htaccess.t ................ ok
t/01_basic.t ................... ok
All tests successful.
Files=20, Tests=162,  7 wallclock secs ( 0.10 usr  0.03 sys +  5.06 cusr 
  0.84 csys =  6.03 CPU)
Result: PASS

Are the test order dependent? That seems to be the difference between 
the 2 runs.

-- 
Michael Peters
Plus Three, LP



------------------------------

Message: 7
Date: Fri, 12 Aug 2011 07:46:46 +1000
From: Gavin Carr <ga...@openfusion.com.au>
Subject: Re: [modauthtkt-users] Query arg separator in generated URLs
To: Michael Peters <mpet...@plusthree.com>
Cc: mod_auth_tkt list <modauthtkt-users@lists.sourceforge.net>
Message-ID: <20110811214646.gb13...@openfusion.com.au>
Content-Type: text/plain; charset=us-ascii; format=flowed

On Thu, Aug 11, 2011 at 10:03:39AM -0400, Michael Peters wrote:
>On 08/11/2011 03:18 AM, Gavin Carr wrote:
>
>> Sure, I guess, if it's causing issues. Though I don't quite see where
>> Facebook is fitting into your request flow Michael? Can you elaborate?
>
>It's not intentionally fitting into our request flow :) We run a lot of
>different sites and our clients can set up protected portions of their
>sites and behind the scenes our code generates mod_auth_tktk .htaccess
>files, etc. Some of our client's members sometimes shared protected
>articles on facebook, etc with other members. Facebook doesn't just let
>people click through links, they are passed through their own tracking
>system and their system is making of mess of links with query strings
>that have semi-colons.
>
>So even if the other person who sees it on facebook is a member they
>can't see the content because it gets mangled by facebook.
>
>Facebook is just the latest example for us of a third party site getting
>those kinds of query strings wrong. And they're probably the biggest
>which is why we've been able to ignore it until now. But some of our
>clients are starting to complain even if it is facebook's fault.

Lovely explanation, thanks. :-)

Cheers,
Gavin




------------------------------

------------------------------------------------------------------------------
Get a FREE DOWNLOAD! and learn more about uberSVN rich system, 
user administration capabilities and model configuration. Take 
the hassle out of deploying and managing Subversion and the 
tools developers use with it. 
http://p.sf.net/sfu/wandisco-dev2dev

------------------------------

_______________________________________________
modauthtkt-users mailing list
modauthtkt-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/modauthtkt-users


End of modauthtkt-users Digest, Vol 26, Issue 2
***********************************************

Reply via email to