[cvs] commit: horde login.php framework/Auth Auth.php gollem redirect.php gollem/lib Gollem.php imp login.php redirect.php imp/lib IMP.php horde/lib Horde.php mimp login.php rakim/guest index.php

Michael M Slusarz slusarz at bigworm.colorado.edu
Wed Jan 28 11:28:14 PST 2004


slusarz     2004/01/28 11:28:12 PST

  Modified files:
    .                    login.php 
    Auth                 Auth.php 
    .                    redirect.php 
    lib                  Gollem.php 
    .                    login.php redirect.php 
    lib                  IMP.php 
    lib                  Horde.php 
    .                    login.php 
    guest                index.php 
  Log:
  Store authentication error reason/messages in a global array ($__autherrors).
  This is a bit messy but necessary since we a) use Auth:: both statically and
  via objects all over the place so there is no guarantee that
  storing something in an object/static variable will be able to be accessed
  elsewhere, and b) since we use Horde::authenticationErrorRedirect() to redirect
  on most authentication problems (like session timeouts), or our logout
  redirects are disjointed in code a long way from the actual authentication
  failures, it starts to get really messy if we were to start explicitly
  explicitly passing around specific Auth object variables all over the place.
  This change has the additional benefit of allowing us to call
  Auth::addLogoutParameters() with just the URL argument and having that
  function automatically generate the correct parameters necessary to output
  the error message on the login page. That function will still allow a
  specific reason to be passed in if the calling code knows exactly what kind
  of error it wants to throw.
  
  Revision  Changes    Path
  1.115     +32 -45    framework/Auth/Auth.php
  1.113     +13 -14    gollem/lib/Gollem.php
  1.32      +4 -4      gollem/redirect.php
  1.372     +3 -2      horde/lib/Horde.php
  2.151     +2 -2      horde/login.php
  1.414     +4 -4      imp/lib/IMP.php
  2.202     +2 -2      imp/login.php
  1.95      +3 -4      imp/redirect.php
  1.17      +2 -2      mimp/login.php
  1.18      +2 -2      rakim/guest/index.php

  Chora Links:
  http://cvs.horde.org/diff.php/framework/Auth/Auth.php?r1=1.114&r2=1.115&ty=u
  http://cvs.horde.org/diff.php/gollem/lib/Gollem.php?r1=1.112&r2=1.113&ty=u
  http://cvs.horde.org/diff.php/gollem/redirect.php?r1=1.31&r2=1.32&ty=u
  http://cvs.horde.org/diff.php/horde/lib/Horde.php?r1=1.371&r2=1.372&ty=u
  http://cvs.horde.org/diff.php/horde/login.php?r1=2.150&r2=2.151&ty=u
  http://cvs.horde.org/diff.php/imp/lib/IMP.php?r1=1.413&r2=1.414&ty=u
  http://cvs.horde.org/diff.php/imp/login.php?r1=2.201&r2=2.202&ty=u
  http://cvs.horde.org/diff.php/imp/redirect.php?r1=1.94&r2=1.95&ty=u
  http://cvs.horde.org/diff.php/mimp/login.php?r1=1.16&r2=1.17&ty=u
  http://cvs.horde.org/diff.php/rakim/guest/index.php?r1=1.17&r2=1.18&ty=u


More information about the cvs mailing list