Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update rubygems and bundler with master branch #4789

Merged
merged 101 commits into from Aug 31, 2021
Merged

Update rubygems and bundler with master branch #4789

merged 101 commits into from Aug 31, 2021

Commits on Aug 31, 2021

  1. Configuration menu
    Copy the full SHA
    9e7b64f View commit details
    Browse the repository at this point in the history
  2. [rubygems/rubygems] Remove unnecessary check

    This error can only be raised when loading the cache, and we only load
    the cache if this condition is met.
    
    rubygems/rubygems@86d692edb8
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    a7f7f64 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    19cd553 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    feefdf4 View commit details
    Browse the repository at this point in the history
  5. [rubygems/rubygems] Show all missing gems when using a bundle before …

    …installing it
    
    Not only the first one that's missing.
    
    This also allows us to simplify things.
    
    rubygems/rubygems@69718a9509
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    ebe10c2 View commit details
    Browse the repository at this point in the history
  6. [rubygems/rubygems] Remove unnecessary exception

    If we have succeeded to materialize the bundle, all specs should be
    loaded.
    
    rubygems/rubygems@bd3712d824
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    8477f38 View commit details
    Browse the repository at this point in the history
  7. [rubygems/rubygems] Remove unnecessary escape sequences

    Bundler formatters already take care of this.
    
    rubygems/rubygems@c24415fdd5
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    67bd725 View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    a4b3b17 View commit details
    Browse the repository at this point in the history
  9. [rubygems/rubygems] Remove unnecessary test repository

    These gems are built and installed to system directly as default gems.
    There's no need to also build a remote repo.
    
    rubygems/rubygems@ad9dad4c22
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    5079061 View commit details
    Browse the repository at this point in the history
  10. [rubygems/rubygems] This spec can pass now on ruby 3

    TSort was released as a library so we can install it, and also other
    gems that are loaded by the spec. Also, Ruby on Windows apparently loads
    fiddle 1.0.6, so we need to also install that to make that not fail.
    
    rubygems/rubygems@2b8dcab99e
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    89e8e2e View commit details
    Browse the repository at this point in the history
  11. [rubygems/rubygems] Fix standalone install of default gems

    Rubygems source replacement was broken.
    
    rubygems/rubygems@3549c122f6
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    a21aa8e View commit details
    Browse the repository at this point in the history
  12. [rubygems/rubygems] Completely avoid replacing sources when in multis…

    …ource compatibility mode
    
    Since this mode is only enabled in frozen mode, it's fine to use the
    lockfile and means we don't have to "prepare" the replacement for
    materialization.
    
    rubygems/rubygems@dda01b288e
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    aa14cb0 View commit details
    Browse the repository at this point in the history
  13. [rubygems/rubygems] We can now avoid source materialization work

    No need to fill up missing names from sources anymore since they should
    now be properly set up correctly from the beginning.
    
    rubygems/rubygems@706fd28681
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    9340a87 View commit details
    Browse the repository at this point in the history
  14. Configuration menu
    Copy the full SHA
    5b1fc08 View commit details
    Browse the repository at this point in the history
  15. [rubygems/rubygems] Don't sort materialized specs when not necessary

    This should fix a weird flaky spec failure, given that the code
    producing the error will be no longer run.
    
    rubygems/rubygems@a171965409
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    e9b524b View commit details
    Browse the repository at this point in the history
  16. [rubygems/rubygems] Fix unnecessary $LOAD_PATH modification in standa…

    …lone script
    
    Due to the `bundler` gem itself being ignored from `$LOAD_PATHS`, a
    `nil` value is being introduced here, resulting in the current folder
    being added to the `$LOAD_PATH` by the standalone script.
    
    I'm pretty sure this is unintentional.
    
    rubygems/rubygems@df54b07b5e
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    d6f6fc1 View commit details
    Browse the repository at this point in the history
  17. [rubygems/rubygems] Remove unnecessary path local variable

    We can use `__dir__` directly.
    
    rubygems/rubygems@0e6083ca94
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    dfea133 View commit details
    Browse the repository at this point in the history
  18. [rubygems/rubygems] Remove unnecessary ruby_engine local variable

    Under some case, this variable might not end up being used, in which
    case running the script would print unused variable warnings.
    
    rubygems/rubygems@a2d6392ada
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    cf9b3b2 View commit details
    Browse the repository at this point in the history
  19. Configuration menu
    Copy the full SHA
    8117dc3 View commit details
    Browse the repository at this point in the history
  20. [rubygems/rubygems] Remove unnecessary ruby_version local variable

    Under some case, this variable might not end up being used, in which
    case running the script would print unused variable warnings.
    
    rubygems/rubygems@bf96030362
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    4e06ab7 View commit details
    Browse the repository at this point in the history
  21. [rubygems/rubygems] Fix standalone generated script to deal with path…

    … sources
    
    In the case of path sources, the path the source is pointing to should
    be added directly to the `$LOAD_PATH` without any modifications.
    
    rubygems/rubygems@d3bba936f0
    
    Co-authored-by: Daniel Niknam <mhmd.niknam@gmail.com>
    2 people authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    457725d View commit details
    Browse the repository at this point in the history
  22. [rubygems/rubygems] Remove redundant part of error message

    It doesn't really add much, in my opinion. We want to be helpful, but
    also concise when possible.
    
    rubygems/rubygems@9d56009cf7
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    fda5259 View commit details
    Browse the repository at this point in the history
  23. [rubygems/rubygems] Simplify error message building in resolver

    Since all requirements have an explicit source now (even if it's the
    default source).
    
    rubygems/rubygems@2c341cfc22
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    937284c View commit details
    Browse the repository at this point in the history
  24. Configuration menu
    Copy the full SHA
    95e3b68 View commit details
    Browse the repository at this point in the history
  25. [rubygems/rubygems] Improve "gem not found in source" errors

    When printing sources inside these error messages, it's useful to only
    consider the current state of the source. For example, when requiring
    `bundler/setup`, the source shouldn't be configured to be able to hit
    the network, so the error message should only mention "locally installed
    gems" to make that more clear.
    
    rubygems/rubygems@30eb14f853
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    d1367dd View commit details
    Browse the repository at this point in the history
  26. Configuration menu
    Copy the full SHA
    b48e5fc View commit details
    Browse the repository at this point in the history
  27. [rubygems/rubygems] The --local flag to bundle install shouldn't …

    …hit the network
    
    If the cache was missing, `bundler` would try to re-fetch it. With the
    `--local` flag, it should just look at installed gems.
    
    rubygems/rubygems@630d29c69e
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    f44a052 View commit details
    Browse the repository at this point in the history
  28. Configuration menu
    Copy the full SHA
    439ab71 View commit details
    Browse the repository at this point in the history
  29. Configuration menu
    Copy the full SHA
    ba14825 View commit details
    Browse the repository at this point in the history
  30. [rubygems/rubygems] Exclude gemspec file itself from gem

    The processed YML data is included as metadata, the source gemspec
    file is unused and just confusing.
    
    rubygems/rubygems@f444478eac
    nobu authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    75598a1 View commit details
    Browse the repository at this point in the history
  31. Configuration menu
    Copy the full SHA
    11c8ecf View commit details
    Browse the repository at this point in the history
  32. Configuration menu
    Copy the full SHA
    b0d4c7b View commit details
    Browse the repository at this point in the history
  33. Configuration menu
    Copy the full SHA
    0e1b49f View commit details
    Browse the repository at this point in the history
  34. Configuration menu
    Copy the full SHA
    fef55c4 View commit details
    Browse the repository at this point in the history
  35. Configuration menu
    Copy the full SHA
    ffd3655 View commit details
    Browse the repository at this point in the history
  36. [rubygems/rubygems] Remove bad TODO

    The commented out code causes failing tests because it's not correct.
    
    rubygems/rubygems@38f0eca91a
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    6cfc924 View commit details
    Browse the repository at this point in the history
  37. [rubygems/rubygems] Remove misleading comment

    When I read, I thought the assertion was incorrect. It's doing the right
    thing, though.
    
    rubygems/rubygems@e72c27367a
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    1aaf94d View commit details
    Browse the repository at this point in the history
  38. Configuration menu
    Copy the full SHA
    9bb53ea View commit details
    Browse the repository at this point in the history
  39. Configuration menu
    Copy the full SHA
    907a2e1 View commit details
    Browse the repository at this point in the history
  40. [rubygems/rubygems] Use Gem.use_gemdeps only from binstubs

    The previous behavior was to automatically require `bundler/setup`
    everytime `rubygems` was required, which I think was too much.
    
    rubygems/rubygems@b25379a295
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    5e5a53f View commit details
    Browse the repository at this point in the history
  41. [rubygems/rubygems] Move list of exceptions to the rescue clause

    Instead of having to match classes and re-raise in the exception body.
    
    rubygems/rubygems@5edf74b781
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    03a1b62 View commit details
    Browse the repository at this point in the history
  42. [rubygems/rubygems] Simplify rescue of bundler errors

    As far as I understand, this block should only be run when
    `bundler/setup` fails. The only other case where these errors could be
    run is when bundler itself is required.
    
    If bundler itself fails to be required or activated (like in old rubies
    where it was not a default gem, for example), the raw error is much more
    helpful than this message.
    
    So we can move the rescue after bundler is succesfully required, and
    that simplifies the list of exceptions that we need to track to just
    `Bundler::Error`.
    
    rubygems/rubygems@3663c11e93
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    019cbe1 View commit details
    Browse the repository at this point in the history
  43. Configuration menu
    Copy the full SHA
    baec0df View commit details
    Browse the repository at this point in the history
  44. [rubygems/rubygems] Avoid double CI runs in default Github Actions ge…

    …nerated config
    
    Otherwise CI will run once for every pull request update and once for
    every push.
    
    The disavantage is that people working on a non default branch without
    creating pull requests won't get CI run at all.
    
    I think the advantages are greater than the disadvantages, it's also my
    personal workflow, and it's something desired enough so that someone
    opened an issue about it.
    
    So let's do it!
    
    rubygems/rubygems@924d916a73
    
    Co-authored-by: NeimadTL <damientalbot26@gmail.com>
    2 people authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    fa70421 View commit details
    Browse the repository at this point in the history
  45. [rubygems/rubygems] Get a CI matrix configured in default Github Acti…

    …ons generated config
    
    Even if it only has one entry at the moment, it makes it easier to add
    new entries by doing it this way.
    
    rubygems/rubygems@46232fe265
    
    Co-authored-by: NeimadTL <damientalbot26@gmail.com>
    2 people authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    f538329 View commit details
    Browse the repository at this point in the history
  46. Configuration menu
    Copy the full SHA
    f1d2238 View commit details
    Browse the repository at this point in the history
  47. Configuration menu
    Copy the full SHA
    d34b4d3 View commit details
    Browse the repository at this point in the history
  48. [rubygems/rubygems] Expect the right permissions on Windows

    Given Windows doesn't have executable bit.
    
    rubygems/rubygems@35dc3fa845
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    ef432b5 View commit details
    Browse the repository at this point in the history
  49. [rubygems/rubygems] Remove RUBYGEMS_GEMDEPS warning

    When setting the `RUBYGEMS_GEMDEPS` environment variable to allow
    skipping `bundle exec`, `bundler` will print a warning about potential
    incompatibility.
    
    Initially the `RUBYGEMS_GEMDEPS` variable used a completely different
    (re)implementation of `bundler` functionality. That implementation was
    not battle tested and could potentially differ in behaviour from what
    `bundler` does. That's why print a warning.
    
    However, these days, all `rubygems` does when `RUBYGEMS_GEMDEPS` is set
    is to require `bundler/setup`, so there's no risk of any
    incompatibility, since that's just plain `bundler`.
    
    rubygems/rubygems@bbddc27016
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    a61c5ae View commit details
    Browse the repository at this point in the history
  50. [rubygems/rubygems] Disable RUBYGEMS_GEMDEPS for bundler spec run

    Running `bundler` specs using `bundler` is not supported.
    
    rubygems/rubygems@cc97b6773d
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    ab1ab97 View commit details
    Browse the repository at this point in the history
  51. Configuration menu
    Copy the full SHA
    975aaab View commit details
    Browse the repository at this point in the history
  52. Configuration menu
    Copy the full SHA
    a1c0e8b View commit details
    Browse the repository at this point in the history
  53. [rubygems/rubygems] Enhance error handling when loading the rubygems/…

    …defaults/operating_system file
    
    When loading `rubygems/defaults/operating_system`
    - we want to keep it silent if the raised exception is a LoadError
    - we want to print a message in other cases and ask users to report the issue to their OS support.
    
    Ruby 3 comes with special error handling for loading `rubygems` and it will show a warning when LoadError exception raised for requiring 'rubygem'.
    Because of that, we decided to leave the LoadError scenario as it is.
    Reference: https://github.com/ruby/ruby/blob/d1998d8767affe58be0bd09ec536dae9198a7fbd/gem_prelude.rb#L1-L5
    
    rubygems/rubygems@0a97e12fe1
    daniel-niknam authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    962ba36 View commit details
    Browse the repository at this point in the history
  54. [rubygems/rubygems] Exclude truffleruby from the test for failure of …

    …operating_system.rb
    
    truffleruby does not raise any error when requiring `rubygems`.
    
    rubygems/rubygems@21e2fcd0b2
    daniel-niknam authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    58dc854 View commit details
    Browse the repository at this point in the history
  55. Configuration menu
    Copy the full SHA
    6abe90e View commit details
    Browse the repository at this point in the history
  56. Configuration menu
    Copy the full SHA
    1091997 View commit details
    Browse the repository at this point in the history
  57. [rubygems/rubygems] Reword the message

    For errors that could happened while loading `rubygems/defaults/operating_system`
    
    rubygems/rubygems@6e1e2141f8
    daniel-niknam authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    d10567e View commit details
    Browse the repository at this point in the history
  58. [rubygems/rubygems] Replace StandardError with Exception

    We think it's unlikely that `rubygems/defaults/operating_system` could be shipped with a SyntaxError so StandardError could be better choice to prevent "false positives" errors.
    
    rubygems/rubygems@1f73e784dd
    daniel-niknam authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    026d743 View commit details
    Browse the repository at this point in the history
  59. [rubygems/rubygems] Remove MacOS specific gem layout

    MacOS should properly configure Ruby. They should not expect us to
    maintain a different layout just for them.
    
    rubygems/rubygems@ecad900925
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    9157a75 View commit details
    Browse the repository at this point in the history
  60. [rubygems/rubygems] Remove MacOS specific extra GEM_PATH

    They should properly configure `GEM_PATH` instead.
    
    rubygems/rubygems@3bd9ae33ca
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    e2165d3 View commit details
    Browse the repository at this point in the history
  61. Configuration menu
    Copy the full SHA
    9662d71 View commit details
    Browse the repository at this point in the history
  62. Configuration menu
    Copy the full SHA
    07274af View commit details
    Browse the repository at this point in the history
  63. [rubygems/rubygems] Fix some failing Bundler tests with old Git.

    Use the `git branch --list` rather than the `git branch -l` for better
    compatibility. Because the `git branch -l` is used to create a new branch in
    Git version < 2.20.0.
    
    rubygems/rubygems@eac5be7d06
    junaruga authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    88eb937 View commit details
    Browse the repository at this point in the history
  64. [rubygems/rubygems] Restore working bundle check behaviour

    As part of a recent bug fix where bundler was accidentally hitting the
    network when not supposed to, I made some refactoring, and the commit I'm
    reverting here
    (rubygems/rubygems@d74830d)
    was some cleanup that those refactorings allowed according to "past me".
    
    That was completely wrong, `bundle check` should never consider cached
    gems, only installed gems, so the code that was removed was necessary.
    
    rubygems/rubygems@5483e98305
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    c85835c View commit details
    Browse the repository at this point in the history
  65. Configuration menu
    Copy the full SHA
    f2170cc View commit details
    Browse the repository at this point in the history
  66. [rubygems/rubygems] Fix bundle check showing duplicated gems

    If the lockfile contains multiple platforms, `bundle check` would show
    duplicated missing gems.
    
    rubygems/rubygems@6ac5931783
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    dcd12c9 View commit details
    Browse the repository at this point in the history
  67. Configuration menu
    Copy the full SHA
    e9c4ab4 View commit details
    Browse the repository at this point in the history
  68. [rubygems/rubygems] Pick the last founded gems when fetching metadata

    rubygems/rubygems@e3d150d822
    
    Co-authored-by: David Rodríguez <deivid.rodriguez@riseup.net>
    2 people authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    ba5e260 View commit details
    Browse the repository at this point in the history
  69. [rubygems/rubygems] Refactor Ruby platform priority condition to its …

    …own method
    
    The `Gem::Platform::RUBY ? -1 : 1` has been used multiple times in different places and could be refactored to a method (DRY).
    
    rubygems/rubygems@9d43ca8f0c
    daniel-niknam authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    b26dd99 View commit details
    Browse the repository at this point in the history
  70. Configuration menu
    Copy the full SHA
    367a801 View commit details
    Browse the repository at this point in the history
  71. Configuration menu
    Copy the full SHA
    29d019d View commit details
    Browse the repository at this point in the history
  72. [rubygems/rubygems] Make plugin installation idempotent

    The error had not be caught be specs because `bundle install` was
    returning a zero exit code when plugin installation errors happened. So
    I fixed that issue too.
    
    rubygems/rubygems@90cde87856
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    5bf9c15 View commit details
    Browse the repository at this point in the history
  73. Configuration menu
    Copy the full SHA
    e96d21e View commit details
    Browse the repository at this point in the history
  74. Configuration menu
    Copy the full SHA
    d435d64 View commit details
    Browse the repository at this point in the history
  75. Configuration menu
    Copy the full SHA
    cc96c9f View commit details
    Browse the repository at this point in the history
  76. Configuration menu
    Copy the full SHA
    5189579 View commit details
    Browse the repository at this point in the history
  77. [rubygems/rubygems] Move Bundler.rubygems.clear_paths to a better p…

    …lace
    
    Since it clears both home and path.
    
    rubygems/rubygems@fadf5e03ea
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    3075d6c View commit details
    Browse the repository at this point in the history
  78. Configuration menu
    Copy the full SHA
    5d2c6c8 View commit details
    Browse the repository at this point in the history
  79. Configuration menu
    Copy the full SHA
    b30e162 View commit details
    Browse the repository at this point in the history
  80. Configuration menu
    Copy the full SHA
    02fbf0f View commit details
    Browse the repository at this point in the history
  81. [rubygems/rubygems] Introduce Gem::PrintableUri that would redact U…

    …RIs to be used on outputs
    
    We need to redact URI credential in several places and copy pasting the code into each part of it is not ideal. This class is responsible for parsing URI strings and redacting credential from it. Also, it will handle URI object in the same manner. We will be reusing this class whenever we need to print/display a URI to users.
    URI with the following format will be redacted:
    - Token: `http://my-secure-token@example.com` => `http://REDACTED@example.com`
    - Username & Password: `http://my-username:my-secure-password@example.com` => `http://my-username:REDACTED@example.com`
    - x-oauth-basic: `http://my-secure-token:x-oauth-basic@example.com` => `http://REDACTED:x-oauth-basic@example.com`
    
    rubygems/rubygems@f1e45d3a89
    daniel-niknam authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    3e3fa81 View commit details
    Browse the repository at this point in the history
  82. [rubygems/rubygems] Using Gem::PrintableUri in `Gem::Commands::Inst…

    …allCommand` class
    
    The `x.source.uri` could be a source URI with a credential. Using `Gem::PrintableUri` to make sure we are redacting sensitive information from it.
    
    rubygems/rubygems@8755ee0aaa
    daniel-niknam authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    0daaed9 View commit details
    Browse the repository at this point in the history
  83. [rubygems/rubygems] Using Gem::PrintableUri in Gem::Request class

    The `@uri` variable could be a source URI with a credential. Using `Gem::PrintableUri` to make sure we are redacting sensitive information from it when logging on verbose mode.
    
    rubygems/rubygems@f566787211
    daniel-niknam authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    8e49b24 View commit details
    Browse the repository at this point in the history
  84. [rubygems/rubygems] Refactor Gem::RemoteFetcher::FetchError initial…

    …izer to `build` method
    
    The `initialize` method is already doing a lot and by adding the `Gem::PrintableUri` to redact sensitive information, things are getting complicated and hard to read here. For the start, I have refactored the `initialize` method into a class method called `build`.
    
    rubygems/rubygems@4312e8fdf5
    daniel-niknam authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    0f3f5b3 View commit details
    Browse the repository at this point in the history
  85. Configuration menu
    Copy the full SHA
    ddcce7c View commit details
    Browse the repository at this point in the history
  86. Configuration menu
    Copy the full SHA
    a924382 View commit details
    Browse the repository at this point in the history
  87. Configuration menu
    Copy the full SHA
    6a4591d View commit details
    Browse the repository at this point in the history
  88. Configuration menu
    Copy the full SHA
    3d9ef52 View commit details
    Browse the repository at this point in the history
  89. [rubygems/rubygems] Remove ruby/uri requirement

    For the purpose of this class, we need to make sure the return object by `Gem::UriParser.parse_uri` method will have the following method:
    - user
    - user=
    - password
    - password=
    So we  can remove the the `uri` dependency and just look for the methods to exist.
    
    rubygems/rubygems@241e093597
    daniel-niknam authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    62555dd View commit details
    Browse the repository at this point in the history
  90. Configuration menu
    Copy the full SHA
    4d30fa1 View commit details
    Browse the repository at this point in the history
  91. Configuration menu
    Copy the full SHA
    2b2d821 View commit details
    Browse the repository at this point in the history
  92. Configuration menu
    Copy the full SHA
    0275212 View commit details
    Browse the repository at this point in the history
  93. Configuration menu
    Copy the full SHA
    5c4cbb7 View commit details
    Browse the repository at this point in the history
  94. Configuration menu
    Copy the full SHA
    e98c95e View commit details
    Browse the repository at this point in the history
  95. [rubygems/rubygems] Remove unnecessary tests

    We will be passing string to this class so there is no need to be defensive about it.
    
    rubygems/rubygems@30bd52a977
    daniel-niknam authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    fa86c0c View commit details
    Browse the repository at this point in the history
  96. Configuration menu
    Copy the full SHA
    f8ebe51 View commit details
    Browse the repository at this point in the history
  97. Configuration menu
    Copy the full SHA
    69932dd View commit details
    Browse the repository at this point in the history
  98. Configuration menu
    Copy the full SHA
    00a7fb7 View commit details
    Browse the repository at this point in the history
  99. [rubygems/rubygems] Merge Gem::UriParser and Gem::PrintableUri in…

    …to a `Gem::Uri` class
    
    The new class is a wrapper on top of an URI. And then, when you want
    credentials redacted, you call `#redacted` that returns a copy of itself,
    but with credentials redacted.
    
    rubygems/rubygems@9581c2740a
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    1d8e1aa View commit details
    Browse the repository at this point in the history
  100. [rubygems/rubygems] Remove syck traces from bundler

    Same reason as in the previous commit.
    
    rubygems/rubygems@f00a6c8516
    deivid-rodriguez authored and hsbt committed Aug 31, 2021
    Configuration menu
    Copy the full SHA
    c1d443d View commit details
    Browse the repository at this point in the history
  101. Configuration menu
    Copy the full SHA
    e0a46ba View commit details
    Browse the repository at this point in the history