Skip to content
Permalink
Browse files
#214 - removes references to request->connection->client_ip
  • Loading branch information
Sebastien Francois committed May 30, 2014
1 parent 575cee9 commit 26e97fc3dbaa28e89e7ffbe0e6f8eedbfc7804cd
Showing with 13 additions and 12 deletions.
  1. +4 −4 lib/defaultcfg/cfg.d/security.pl
  2. +1 −1 perl_lib/EPrints/DataObj/LoginTicket.pm
  3. +8 −7 perl_lib/EPrints/MetaField/Recaptcha.pm
@@ -36,14 +36,14 @@
return( "ALLOW" ) unless( $request->has_expired() );
}
}
}

my $ip = $r->connection()->remote_ip();

}

# Example of how to allow an override for certain basic auth type usernames/passwords.
# This is useful if you want the site to be read by a crawler, for example.
# You may wish to wrap it all in a if( $ip eq "xxx" ) for added security.
#
# my $ip = $doc->repository->remote_ip();
#
# my( $res, $passwd_sent ) = $r->get_basic_auth_pw;
# my( $user_sent ) = $r->user;
# if( defined $user_sent )
@@ -140,7 +140,7 @@ sub new_from_request

my $dataset = $repo->dataset( $class->get_dataset_id );

my $ip = $r->connection->remote_ip;
my $ip = $repo->remote_ip;

my $ticket;

@@ -89,16 +89,17 @@ sub render_input_field_actual

sub form_value_actual
{
my( $self, $session, $object, $basename ) = @_;
my( $self, $repo, $object, $basename ) = @_;

my $private_key = $session->config( "recaptcha", "private_key" );
my $remote_ip = $session->get_request->connection->remote_ip;
my $challenge = $session->param( "recaptcha_challenge_field" );
my $response = $session->param( "recaptcha_response_field" );
my $private_key = $repo->config( "recaptcha", "private_key" );

my $remote_ip = $repo->remote_ip;
my $challenge = $repo->param( "recaptcha_challenge_field" );
my $response = $repo->param( "recaptcha_response_field" );

if( !defined $private_key )
{
$session->get_repository->log( "recaptcha private_key not set" );
$repo->log( "recaptcha private_key not set" );
return undef;
}

@@ -133,7 +134,7 @@ sub form_value_actual

# error talking to recaptcha, so lets continue to avoid blocking the user
# in case of network problems
$session->get_repository->log( "Error contacting recaptcha: ".$r->code." ".$r->message );
$repo->log( "Error contacting recaptcha: ".$r->code." ".$r->message );

return undef;
}

0 comments on commit 26e97fc

Please sign in to comment.