Skip to content

Commit

Permalink
Merge pull request #8638 from mlschroe/appx
Browse files Browse the repository at this point in the history
[backend] Fix appx signing with special certificate
  • Loading branch information
mlschroe committed Oct 30, 2019
2 parents 8703a56 + a45b48b commit 3aae1fe
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/backend/bs_signer
Original file line number Diff line number Diff line change
Expand Up @@ -340,7 +340,7 @@ sub rsasign {
sub signappx {
my ($signfile, $jobdir, $projid, $signkey, $cert, @signargs) = @_;

if ((!$signkey || length($signkey) < 2) && $BSConfig::sign_type) {
if ((!$signkey || length($signkey) <= 2) && $BSConfig::sign_type) {
# special signkey, ask for cert again with signtype appx
my $param = {
'uri' => "$BSConfig::srcserver/getsslcert",
Expand Down
2 changes: 1 addition & 1 deletion src/backend/bs_worker
Original file line number Diff line number Diff line change
Expand Up @@ -1101,7 +1101,7 @@ sub getsslcert {
$cert = BSRPC::rpc({
'uri' => "$server/getsslcert",
'timeout' => $gettimeout,
}, undef, "project=$buildinfo->{'project'}", "autoextend=1");
}, undef, @args);
};
die("could not retrieve ssl certificate: $@\n") if $@;
writestr("$dir/_projectcert.crt", undef, $cert) if $cert;
Expand Down

0 comments on commit 3aae1fe

Please sign in to comment.