From ebf8fd8065ef9cd8477ecf832666153f1522ad4b Mon Sep 17 00:00:00 2001 From: Martin Kamerbeek Date: Tue, 11 Jul 2006 16:09:13 +0000 Subject: [PATCH] fix: Session id --- docs/changelog/7.x.x.txt | 1 + lib/WebGUI/Session.pm | 1 + 2 files changed, 2 insertions(+) diff --git a/docs/changelog/7.x.x.txt b/docs/changelog/7.x.x.txt index aab083128..e27e31b3e 100644 --- a/docs/changelog/7.x.x.txt +++ b/docs/changelog/7.x.x.txt @@ -5,6 +5,7 @@ list - fix: Syndicated wobject erro 6.8+ - fix: new spectre.pl error (Martin Kamerbeek / Procolix) + - fix: Session id (Martin Kamerbeek / Procolix) 7.0.0 - Welcome to a whole new world of WebGUI. After 2.5 years and 20,000 hours of diff --git a/lib/WebGUI/Session.pm b/lib/WebGUI/Session.pm index f0501ad90..6e5b6e834 100644 --- a/lib/WebGUI/Session.pm +++ b/lib/WebGUI/Session.pm @@ -370,6 +370,7 @@ sub open { bless $self , $class; $self->{_request} = Apache2::Request->new($request) if (defined $request); my $sessionId = shift || $self->http->getCookies->{"wgSession"} || $self->id->generate; + $sessionId = $self->id->generate if ($sessionId !~ m/^[A-Za-z0-9\+\/=]{22}$/); my $noFuss = shift; $self->{_var} = WebGUI::Session::Var->new($self,$sessionId, $noFuss); $self->errorHandler->warn("You've disabled cache in your config file and that can cause many problems on a production site.") if ($config->get("disableCache"));