Skip to content

Bump net-protocol from 0.2.2 to 0.3.0 - #3106

Merged
ksss merged 1 commit into
ruby:masterfrom
dak2:bump-net-protocol-0.3.0
Aug 30, 2026
Merged

Bump net-protocol from 0.2.2 to 0.3.0#3106
ksss merged 1 commit into
ruby:masterfrom
dak2:bump-net-protocol-0.3.0

Conversation

@dak2

@dak2 dak2 commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Problem

test (head, stdlib_test rubocop) fails on ruby-head with 8 errors, all in Net::HTTP / open-uri (run):

ArgumentError: wrong number of arguments (given 3, expected 1..2)
  net-protocol-0.2.2/lib/net/protocol.rb:194:in 'Net::BufferedIO#readuntil'
  ruby-head/lib/ruby/4.1.0+4/net/http/response.rb:164:in 'Net::HTTPResponse.read_line'

Cause

ruby/net-http#325 (in ruby/ruby as ruby/ruby@331568b2a) made Net::HTTPResponse use readuntil(limit:), which requires net-protocol >= 0.3.0.

ruby-head ships 0.3.0 as a default gem, so it is consistent on its own. But net/http is a default gem too, so its dependency never reaches Bundler's resolution — our lock pinned 0.2.2 through net-smtp, which carries no lower bound, and the installed 0.2.2 shadowed ruby-head's 0.3.0.

🤖 Generated with Claude Code

ruby-head now ships a net/http that calls net-protocol's
readuntil(limit:), which needs 0.3.0. Our lock pinned 0.2.2 and it
shadowed ruby-head's default gem, breaking the Net::HTTP stdlib tests:
https://github.com/ruby/rbs/actions/runs/33056526924/job/98464589988

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

@ksss ksss left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ksss
ksss added this pull request to the merge queue Aug 30, 2026
Merged via the queue into ruby:master with commit 58a7361 Aug 30, 2026
23 checks passed
@dak2
dak2 deleted the bump-net-protocol-0.3.0 branch August 30, 2026 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants