Skip to content

Fix the descriptor checks of the Proxy getOwnPropertyDescriptor trap - #1635

Open
saghul wants to merge 1 commit into
masterfrom
worktree-fix-proxy-gopd-compat
Open

Fix the descriptor checks of the Proxy getOwnPropertyDescriptor trap#1635
saghul wants to merge 1 commit into
masterfrom
worktree-fix-proxy-gopd-compat

Conversation

@saghul

@saghul saghul commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Fixes #1629.

@bnoordhuis

Copy link
Copy Markdown
Contributor

Has conflicts. General note: I keep wondering with all these obscure edge case bug reports why test262 doesn't have coverage.

My preference is for people to submit a regression test upstream first. That way we can be sure it's a) an actual corner case and not some misreading of the spec, and b) whether other engines implement it correctly or not.

If not all do, that's probably an indication of something.

@saghul

saghul commented Aug 29, 2026

Copy link
Copy Markdown
Contributor Author

FWIW, I submitted a PR with a bunch of extra coverage for iterators and it got approved reasonably quickly, but it's still pending merge :-/

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.

Proxy getOwnPropertyDescriptor trap accepts an incompatible descriptor for a non-configurable property

2 participants