Fix Seahorse::Client::NetHttp::ConnectionPool::ExtendedSession unused block warning

[bkuhlmann]

Describe the bug

Hello. 👋 I'm getting the following warning in my console:

$HOME/.cache/frum/versions/3.4.1/lib/ruby/3.4.0/delegate.rb:84: warning: the block passed to 'Seahorse::Client::NetHttp::ConnectionPool::ExtendedSession#__getobj__' defined at $HOME/.cache/frum/versions/3.4.1/lib/ruby/gems/3.4.0/gems/aws-sdk-core-3.217.1/lib/seahorse/client/net_http/connection_pool.rb:338 may be ignored

The problem is that a block is being passed to the response when it shouldn't because the response doesn't use a block. This is similar to the issue that was addressed in 3169.

Regression Issue

• Select this option if this issue appears to be a regression.

Expected Behavior

I would expect this error to not show up since the block is never used/implemented in the code.

Current Behavior

See above.

Reproduction Steps

I believe if you add this line to your test suite, you'll see the warning show up immediately:

Warning[:strict_unused_block] = true

Possible Solution

You could change your method signature as follows:

class ExtendedSession < Delegator
  # Truncated for brevity...

  def __getobj__(&)
    @http
  end

  # Truncated for brevity...
end

Use of the anonymous block should correct the issue.

Additional Information/Context

No response

Gem name ('aws-sdk', 'aws-sdk-resources' or service gems like 'aws-sdk-s3') and its version

aws-sdk-core

Environment details (Version of Ruby, OS environment)

ruby 3.4.1 (2024-12-25 revision 48d4efcb85) +YJIT +PRISM [arm64-darwin24.2.0]

[github-actions]

This issue is now closed. Comments on closed issues are hard for our team to see.
If you need more assistance, please open a new issue that references this one.

[richardwang1124]

Hello! Thank you for bringing this issue to our attention. We have merged a fix for this issue and it will be part of tomorrow's release.