Project

General

Profile

Actions

Bug #15384

closed

ssl_certs are duplicated in RubyGems and Bundler

Added by vo.x (Vit Ondruch) over 5 years ago. Updated almost 5 years ago.

Status:
Closed
Target version:
-
ruby -v:
ruby 2.6.0dev (2018-11-29 trunk 66092) [x86_64-linux]
[ruby-core:90313]

Description

It is pity that the same ssl_certs are shipped on multiple places, once as part of RubyGems and the other set as part of Bundler. This makes the security review much harder (actually, in Fedora/RHEL packages, we are not supposed to ship any certificates, so it makes it harder to remove them).

Therefore, please ship just one copy of the certificates if really necessary (it should not be necessary on properly maintained systems).


Files

unify-certification-bundler.patch (14.3 KB) unify-certification-bundler.patch hsbt (Hiroshi SHIBATA), 03/14/2019 02:47 AM

Updated by shevegen (Robert A. Heiler) over 5 years ago

Agree on the "one rather than two". It is probably redundant after the merge.

I can't answer the second sentence since there may have been (different?)
reasons for adding certificates - but it would make sense to require only
one rather than two either way.

Updated by hsbt (Hiroshi SHIBATA) over 5 years ago

  • Status changed from Open to Assigned
  • Assignee set to hsbt (Hiroshi SHIBATA)

Updated by hsbt (Hiroshi SHIBATA) about 5 years ago

I made a patch that unifies both certificates. I propose it to bundler upstream.

Updated by vo.x (Vit Ondruch) about 5 years ago

Is the patch correct? Will it work when RubyGems are updated via gem update --system? I have not tested it, just wondering ...

Moreover, I don't understand why Bundler does not use RubyGems facilities for such functionality (but I understand the patch would be probably more complex :) ).

Updated by vo.x (Vit Ondruch) about 5 years ago

vo.x (Vit Ondruch) wrote:

Is the patch correct? Will it work when RubyGems are updated via gem update --system? I have not tested it, just wondering ...

Gem::RUBYGEMS_DIR should be probably used to initialize the rubygems_certs_dir

https://github.com/rubygems/rubygems/blob/master/lib/rubygems.rb#L116

Updated by vo.x (Vit Ondruch) about 5 years ago

There is even Gem::Request.get_cert_files

Updated by hsbt (Hiroshi SHIBATA) almost 5 years ago

  • Status changed from Assigned to Closed
  • Backport changed from 2.4: UNKNOWN, 2.5: UNKNOWN to 2.4: DONTNEED, 2.5: DONTNEED, 2.6: DONTNEED

I fixed it at r67539

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0Like0Like0Like0