EPrints Technical Mailing List Archive

See the EPrints wiki for instructions on how to join this mailing list and related information.

Message: #00555

< Previous (by date) | Next (by date) > | < Previous (in thread) | Next (in thread) > | Messages - Most Recent First | Threads - Most Recent First

[EP-tech] Screen::Logout bug?

I think there is a problem in perl_lib/EPrints/Plugin/Screen/Logout.pm for sites with multiple secure archives. The upshot is that

 

sub render_action_link

{

        my( $self, %opts ) = @_;

 

        my $link = $self->SUPER::render_action_link( %opts );

        $link->setAttribute( href ="" $self->{session}->config( "http_cgiroot" ) . "/logout" );

 

        return $link;

}

 

should probably check for the existence of https_cgiroot. We’ve bodged the code to

 

sub render_action_link

{

        my( $self, %opts ) = @_;

 

        my $link = $self->SUPER::render_action_link( %opts );

        $link->setAttribute( href ="" $self->{session}->config( "https_cgiroot" ) . "/logout" ); # note the https

 

        return $link;

}

 

which works for us, but which I suspect isn’t the general solution.

 

The background is that we have two secure instances, http://eprints.uwe.ac.uk and http://researchdata.uwe.ac.uk. Unless the change is made to render_action_link the logout link is researchdata.uwe.ac.uk appears as http://eprints.uwe.ac.uk/cgi/logout, which doesn’t work.

 

Thanks,

 

Jon Hallett CEng MBCS CITP

Library IT Manager

Library Services

UWE Bristol

Frenchay Campus

Coldharbour Lane

Bristol

BS16 1QY

+44 (0)117 328 1560

jon.hallett@uwe.ac.uk