summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSam Thursfield <sam.thursfield@codethink.co.uk>2014-12-04 13:14:07 +0000
committerSam Thursfield <sam.thursfield@codethink.co.uk>2014-12-04 13:14:07 +0000
commit8eaaac2c17194299245ae0c52be43f3604f326b9 (patch)
tree6242dc19a81ce66787e50bab9e6bf84b6d5baca6
parent2101ce238f5ecf05a5260486d5af892996f27c68 (diff)
downloadimport-8eaaac2c17194299245ae0c52be43f3604f326b9.tar.gz
rubygems: Detect more signed Gems
The multi_json Gem wasn't being detected as signed, because the 'signing_key' field is an expression that can evaluate to 'nil' in some cases. In this Gem the 'cert_chain' field was still a standard string. Hopefully checking for the presence of either will catch all cases (and false positives should be harmless anyway).
-rwxr-xr-xbaserockimport/exts/rubygems.to_chunk2
1 files changed, 1 insertions, 1 deletions
diff --git a/baserockimport/exts/rubygems.to_chunk b/baserockimport/exts/rubygems.to_chunk
index 323cbb0..f6fe514 100755
--- a/baserockimport/exts/rubygems.to_chunk
+++ b/baserockimport/exts/rubygems.to_chunk
@@ -63,7 +63,7 @@ class RubyGemChunkMorphologyGenerator < Importer::Base
end
def is_signed_gem(spec)
- spec.signing_key != nil
+ spec.signing_key != nil || spec.cert_chain != nil
end
def generate_chunk_morph_for_gem(gemspec_file, spec)