Skip to content

fix(pika): use ObjectProxy for ReadyMessagesDequeProxy to restore iterability with wrapt 2.x (#4461)#4485

Merged
xrmx merged 1 commit into
open-telemetry:release/v1.41.x-0.62bxfrom
xrmx:1411-pika-backport
Apr 24, 2026
Merged

fix(pika): use ObjectProxy for ReadyMessagesDequeProxy to restore iterability with wrapt 2.x (#4461)#4485
xrmx merged 1 commit into
open-telemetry:release/v1.41.x-0.62bxfrom
xrmx:1411-pika-backport

Conversation

@xrmx

@xrmx xrmx commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Description

wrapt 2.x BaseObjectProxy does not proxy __iter__, breaking iteration over the wrapped deque. Switch to ObjectProxy which proxies all dunder methods including __iter__.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A

Does This PR Require a Core Repo Change?

  • Yes. - Link to PR:
  • No.

Checklist:

See contributing.md for styleguide, changelog guidelines, and more.

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

…rability with wrapt 2.x (open-telemetry#4461)

* use ObjectProxy for pika ReadyMessagesDequeProxy to restore iterability

wrapt 2.x BaseObjectProxy does not proxy __iter__, breaking iteration
over the wrapped deque. Switch to ObjectProxy which proxies all dunder
methods including __iter__.

Assisted-by: Claude Opus 4.6

* update changelog with PR number

Assisted-by: Claude Opus 4.6

* fix lint: import ordering and formatting

Assisted-by: Claude Opus 4.6
@xrmx
xrmx requested a review from a team as a code owner April 24, 2026 09:50
@xrmx xrmx added the backport Use during release PRs. Change CORE_REPO_SHA to point to BASE branch name label Apr 24, 2026
@xrmx
xrmx enabled auto-merge (squash) April 24, 2026 09:55
@xrmx
xrmx merged commit b983adc into open-telemetry:release/v1.41.x-0.62bx Apr 24, 2026
861 checks passed
@github-project-automation github-project-automation Bot moved this to Approved PRs in Python PR digest Apr 24, 2026
@github-project-automation github-project-automation Bot moved this from Approved PRs to Done in Python PR digest Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Use during release PRs. Change CORE_REPO_SHA to point to BASE branch name

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants