GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-146308: Fix error handling issues in _remote_debugging module by pablogsal · Pull Request #146309 · python/cpython

Conversation

@pablogsal

@pablogsal pablogsal commented Mar 22, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member
  • Issue: Fix error handling issues in _remote_debugging module #146308

Some of these issues have been found thanks to a report from @devdanzin ❤️

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@bedevere-app bedevere-app Bot added the awaiting core review label Mar 22, 2026
@bedevere-app bedevere-app Bot mentioned this pull request Mar 22, 2026
Fix error handling issues in _remote_debugging module #146308
Closed
@pablogsal
pablogsal force-pushed the gh-146308-fix-remote-debugging-error-handling branch from 719eb8f to ed3e6b5 Compare March 22, 2026 20:14
@pablogsal
pablogsal requested a review from devdanzin March 22, 2026 20:15
@pablogsal
pythongh-146308: Fix error handling issues in _remote_debugging module
7a0560e
@pablogsal
pablogsal force-pushed the gh-146308-fix-remote-debugging-error-handling branch from 29e2fd6 to 7a0560e Compare March 22, 2026 20:16
brianschubert
brianschubert reviewed Mar 22, 2026
View reviewed changes
Comment thread Misc/NEWS.d/next/Core_and_Builtins/2026-03-22-19-30-00.gh-issue-146308.AxnRVA.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

@pablogsal @brianschubert
Update Misc/NEWS.d/next/Core_and_Builtins/2026-03-22-19-30-00.gh-issu…
ad81c5b
…e-146308.AxnRVA.rst

Co-authored-by: Brian Schubert <[email protected]>
@pablogsal pablogsal added the needs backport to 3.14 bugs and security fixes label Mar 22, 2026
devdanzin
devdanzin reviewed Mar 22, 2026
View reviewed changes
Comment thread Modules/_remote_debugging/binary_io_reader.c Resolved

Uh oh!

There was an error while loading. Please reload this page.

@pablogsal @devdanzin
Update Modules/_remote_debugging/binary_io_reader.c
359cf5c
Co-authored-by: devdanzin <[email protected]>
@pablogsal
pablogsal enabled auto-merge (squash) March 22, 2026 20:46
@pablogsal
pablogsal merged commit ae6adc9 into python:main Mar 22, 2026
50 checks passed

Uh oh!

There was an error while loading. Please reload this page.

@pablogsal
pablogsal deleted the gh-146308-fix-remote-debugging-error-handling branch March 22, 2026 21:12
@bedevere-app bedevere-app Bot removed the awaiting core review label Mar 22, 2026
@miss-islington-app

miss-islington-app Bot commented Mar 22, 2026

Copy link
Copy Markdown

Thanks @pablogsal for the PR 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@miss-islington-app

miss-islington-app Bot commented Mar 22, 2026

Copy link
Copy Markdown

Sorry, @pablogsal, I could not cleanly backport this to 3.14 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker ae6adc907907562e4ffbb5355f12e77e9085c506 3.14
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@miss-islington-app miss-islington-app Bot assigned pablogsal Mar 22, 2026
brianschubert pushed a commit to brianschubert/cpython that referenced this pull request Mar 22, 2026
@pablogsal @brianschubert
[3.14] pythongh-146308: Fix error handling issues in _remote_debuggin…
cbab7f4
…g module (pythonGH-146309)

(cherry picked from commit ae6adc9)

Co-authored-by: Pablo Galindo Salgado <[email protected]>
CuriousLearner added a commit to CuriousLearner/cpython that referenced this pull request Mar 23, 2026
@CuriousLearner
Merge branch 'main' of github.com:python/cpython into fix-pythongh-13…
3e05fa3
…8577

* 'main' of github.com:python/cpython:
  pythongh-146197: Run -m test.pythoninfo on the Emscripten CI (python#146332)
  pythongh-146325: Use `test.support.requires_fork` in test_fastpath_cache_cleared_in_forked_child (python#146330)
  pythongh-146197: Add Emscripten to CI (python#146198)
  pythongh-143387: Raise an exception instead of returning None when metadata file is missing. (python#146234)
  pythongh-108907: ctypes: Document _type_ codes (pythonGH-145837)
  pythongh-146175: Soft-deprecate outdated macros; convert internal usage (pythonGH-146178)
  pythongh-146056: Rework ref counting in treebuilder_handle_end() (python#146167)
  Add a warning about untrusted input to `configparser` docs (python#146276)
  pythongh-145264: Do not ignore excess Base64 data after the first padded quad (pythonGH-145267)
  pythongh-146308: Fix error handling issues in _remote_debugging module (python#146309)
  pythongh-146192: Add base32 support to binascii (pythonGH-146193)
  pythongh-135953: Properly obtain main thread identifier in Gecko Collector (python#146045)
  pythongh-143414: Implement unique reference tracking for JIT, optimize unpacking of such tuples (pythonGH-144300)
  pythongh-146261: Fix bug in `_Py_uop_sym_set_func_version` (pythonGH-146291)
  pythongh-145144: Add more tests for UserList, UserDict, etc (pythonGH-145145)
  pythongh-143959: Fix test_datetime if _datetime is unavailable (pythonGH-145248)
  pythongh-146245: Fix reference and buffer leaks via audit hook in socket module (pythonGH-146248)
  pythongh-140049: Colorize exception notes in `traceback.py` (python#140051)
  Update docs for pythongh-146056 (pythonGH-146213)
@bedevere-app

bedevere-app Bot commented Mar 24, 2026

Copy link
Copy Markdown

GH-146398 is a backport of this pull request to the 3.14 branch.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Mar 24, 2026
pablogsal added a commit that referenced this pull request Mar 25, 2026
@pablogsal
[3.14] gh-146308: Fix error handling issues in _remote_debugging modu…
58c5eda
…le (GH-146309) (#146398)

(cherry picked from commit ae6adc9)
@pablogsal pablogsal mentioned this pull request Apr 4, 2026
gh-130273: Fix traceback color output with unicode characters #142529
Merged
@brianschubert brianschubert mentioned this pull request Apr 12, 2026
remote debugging: read_pylong is missing a PyErr_Format call #148447
Closed
ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
@pablogsal @ljfp
pythongh-146308: Fix error handling issues in _remote_debugging module (
3e5559e
python#146309)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@brianschubert brianschubert brianschubert left review comments
@devdanzin devdanzin devdanzin left review comments

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@pablogsal pablogsal

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@pablogsal @brianschubert @devdanzin
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.