Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork32k
gh-134546: ensure remote pdb script is readable#134552
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Conversation
4fd050e
to855651b
CompareThe news is a bit inaccurate. It's more about client process creating a file that server process does not have access to. It is permission issue, but there are many kinds of permission issues and they are not all fixed. Could you be a little bit more accurate on the news? Also cc@godlygeek and@pablogsal . I think the fix is fine but maybe you have other suggestions. |
maybe it's just a wording thing? I think the news is accurate but perhaps too broad. the script from pdb can't be read in the process that's being debugged |
I think it's a wording thing. I think the issue is that, we did not fix it from the reading side. It's not like we fixed something so the process can read that file. We granted the read permission after we wrote it - that's what's fixed. |
do you have a suggestion for the contents of the news file? |
I would do something like Grant read permission to the attach script for all processes for:func:`pdb.attach` so it can be read by:func:`sys.remote_exec`. I'm also not the expert in news and I'll probably wait for someone else to see if this is a better description. |
Perhaps Ensure:func:`pdb.attach` script is readable by remote Python process. |
Fix looks reasonable to me. For the news entry, maybe "Fix |
@asottile you can sum the suggestions up and make it a bit clearer. news is not as strict as the real documentation. Good to have a clear one but won't be a serious blocker. |
Misc/NEWS.d/next/Library/2025-05-22-18-14-13.gh-issue-134546.fjLVzK.rst OutdatedShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
…jLVzK.rstCo-authored-by: Emma Smith <emma@emmatyping.dev>
Uh oh!
There was an error while loading.Please reload this page.
-m pdb -p
-- other side cannot read debug script due to too-strict permissions #134546