Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DAPPythonDebugger must be extended with new debugpySockets event type #3625

Closed
svenvc opened this issue Feb 13, 2024 · 0 comments
Closed

DAPPythonDebugger must be extended with new debugpySockets event type #3625

svenvc opened this issue Feb 13, 2024 · 0 comments
Assignees

Comments

@svenvc
Copy link
Contributor

svenvc commented Feb 13, 2024

The DebuggerAdaptorProtocol deals with DAPObjects in JSON format going back and forth over stdout. Turns out that on Feb 8, MS released a new version of debugpy which signals a new event type, debugpySockets, which we do not recognise and which blocks the debugger from starting. This looks like an informational event that has something to do with proxies/firewalls.

microsoft/debugpy#1337

The type is an extension to the general DAP protocol, specific to debugpy. We already support 2 of those specific extension events (DAPDebugpyAttachEvent and DAPDebugpyWaitingForServerEvent).
I think we would have to add a new DAPDebugpySocketsEvent. That is not too difficult. The question is how to define this so that it gets processed or even ignored correctly.

@svenvc svenvc self-assigned this Feb 13, 2024
svenvc pushed a commit to feenkcom/pharo-debugadapterprotocol that referenced this issue Feb 13, 2024
@svenvc svenvc closed this as completed Feb 13, 2024
syrel pushed a commit that referenced this issue Feb 13, 2024
Metacello new
    baseline: 'GToolkitForPharo9';
    repository: 'github://feenkcom/gtoolkit:v1.0.520/src';
    load

All commits (including upstream repositories) since last build:
feenkcom/gtoolkit-phlow@4dd438 by Andrei Chi�
Make it possible to customize the selector used to test views in GtPhlowViewTester

feenkcom/gt4pharo@04c8a0 by John Brant
inline message in snippet enhancements

feenkcom/gtoolkit-inspector@cdb66b by Oscar Nierstrasz
Improved layout of examples map using horizontalDominanceTree #3624

feenkcom/PythonBridge@a8bdbb by svenvc
gtoolkit_bridge PyPI version 0.1.2


feenkcom/PythonBridge@f6e5b1 by svenvc
Merge remote-tracking branch 'refs/remotes/origin/main'


feenkcom/PythonBridge@5772c0 by svenvc
tfactorial.py updated to new telemetry API


feenkcom/PythonBridge@9a6320 by svenvc
PBNewPharoPipenvProcess extended to work with DAP server debugger

feenkcom/pharo-debugadapterprotocol@70d508 by svenvc
Merge e70e448b6a8ee16607a12d43937268981a66cc40

feenkcom/pharo-debugadapterprotocol@aba7da by svenvc
DAPDebugpySocketsEvent add [#3625]

feenkcom/lepiter@19483d by Veit Heller
Add an `isForLanguage:` selector to the lepiter code block node

feenkcom/gtoolkit-demos@572874 by Oscar Nierstrasz
Merge 8f9fc055aa6c69a3161260a001beab5c43505968

feenkcom/gtoolkit-demos@95b481 by Oscar Nierstrasz
Started working on complex scenario #3618

feenkcom/gtoolkit-demos@8f9fc0 by Oscar Nierstrasz
Merge branch 'main' of github.com:feenkcom/gtoolkit-demos into main


feenkcom/gtoolkit-demos@56ecdb by Oscar Nierstrasz
started fileout of second scenario


feenkcom/gtoolkit-demos@7ce700 by Oscar Nierstrasz
Worked on EDD scenarios with TG

feenkcom/gtoolkit-demos@3049df by Oscar Nierstrasz
Worked on EDD scenarios with TG

feenkcom/gtoolkit-demos@afbb08 by Oscar Nierstrasz
Merge b6d0419cac933558bdf8daf3892eef92f72bb0d2

feenkcom/gtoolkit-demos@78d67e by Oscar Nierstrasz
EDD example

feenkcom/gtoolkit-demos@b6d041 by Oscar Nierstrasz
EDD prices, first steps


feenkcom/gtoolkit-remoterunner@93bae6 by Alistair Grant
GtRrMachineCountPlot again


feenkcom/gtoolkit-remoterunner@2cee3a by Alistair Grant
GtRrMachineCountPlot: refactor to avoid inherited examples

feenkcom/gtoolkit-remoterunner@22f697 by Alistair Grant
GtRemoteRunner reinstate missing methods

feenkcom/gtoolkit-remoterunner@808d6c by Alistair Grant
Add GtRrRemoteRunnerWorkerMachineSequenceAnalysis and co.

feenkcom/gt4gemstone@b17fd8 by Andrei Chi�
Merge pull request #9 from lifeware-sa/fix_session_button_order

Add order in button of the session

feenkcom/gt4gemstone@fd31ea by Vincent Blondeau
Fixes: [feenkcom/gt4gemstone#8]

ff1f1f by svenvc
Python Debugger Setup & Example page updated more

41d922 by svenvc
Merge 0413d57

b71a06 by svenvc
Python Debugger Setup & Example page updated
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

No branches or pull requests

1 participant