Skip to content

Commit 4c09f5e

Browse files
committed
Fully cover WebAPI::Auth::OpenID
Reference: https://progress.opensuse.org/issues/178948
1 parent b88c0b4 commit 4c09f5e

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

lib/OpenQA/WebAPI/Auth/OpenID.pm

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,8 @@ sub auth_login ($c) {
2222
);
2323

2424
my $claimed_id = $csr->claimed_identity($c->config->{openid}->{provider});
25-
if (!defined $claimed_id) {
26-
log_error("Claiming OpenID identity for URL '$url' failed: " . $csr->err);
27-
return;
28-
}
25+
!defined $claimed_id and do { log_error("Claiming OpenID identity for URL '$url' failed: " . $csr->err); return; };
26+
2927
$claimed_id->set_extension_args(
3028
'http://openid.net/extensions/sreg/1.1',
3129
{
@@ -59,8 +57,8 @@ sub auth_login ($c) {
5957
return_to => $return_url,
6058
trust_root => qq{$url/},
6159
);
62-
return (redirect => $check_url, error => 0) if $check_url;
63-
return (error => $csr->err);
60+
return (error => $csr->err) unless $check_url;
61+
return (redirect => $check_url, error => 0);
6462
}
6563

6664
sub _first_last_name ($ax) { join(' ', $ax->{'value.firstname'} // '', $ax->{'value.lastname'} // '') }
@@ -94,6 +92,7 @@ sub auth_response ($c) {
9492
%params = map { $_ => URI::Escape::uri_unescape($params{$_}) } keys %params;
9593

9694
my $csr = Net::OpenID::Consumer->new(
95+
# uncoverable statement
9796
debug => sub (@args) { $c->app->log->debug('Net::OpenID::Consumer: ' . join(' ', @args)) },
9897
ua => LWP::UserAgent->new,
9998
required_root => $url,

0 commit comments

Comments
 (0)