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

[Feature request] update netcoredbg gadget to support exceptions #448

Closed
sebaFlame opened this issue Sep 6, 2021 · 8 comments · Fixed by #530
Closed

[Feature request] update netcoredbg gadget to support exceptions #448

sebaFlame opened this issue Sep 6, 2021 · 8 comments · Fixed by #530

Comments

@sebaFlame
Copy link

netcoredbg recently added exceptionBreakpointFilters support: netcoredbg commit

this should be available in 1.2.0-825
Unfortunately it doesn't seem to be a drop-in replacement, it breaks breakpoints and I can not configure exceptions filters.

@puremourning
Copy link
Owner

If netcoredbg has regressed, then I'm not going to update to a version that's broken. Maybe report the regression to them ?

@sebaFlame
Copy link
Author

sebaFlame commented Sep 8, 2021

There is also the issue of the exception filters not being sent. From what I understand so far, the filters should get sent when a breakpoint gets set?

.vimspector.json

{
  "adapters": {
    "netcoredbg-debug": {
      "attach": {
        "pidProperty": "processId",
        "pidSelect": "ask"
      },
      "command": [
        "${gadgetDir}/netcoredbg/netcoredbg",
        "--interpreter=vscode",
        "--engineLogging=${workspaceRoot}/netcoredbg.engine.log",
        "--log=${workspaceRoot}/netcoredbg.log"
      ],
      "configuration": {
        "cwd": "${workspaceRoot}"
      },
      "name": "netcoredbg-debug"
    }
  },
  "configurations": {
    "test - netcoredbg": {
      "adapter": "netcoredbg-debug",
      "breakpoints": {
        "exception": {
            "unhandled": "Y",
            "user-unhandled": "Y",
            "throw": "Y",
            "all": "N"
        }
      },
      "configuration": {
        "request": "attach",
	  }
    }
  }
}

"all" & "user-unhandled" are supported

Log for 1.2.0-782 -
_vimspector_log_Vimspector_782.log
Log for 1.2.0-825 -
_vimspector_log_Vimspector_825.log

@puremourning
Copy link
Owner

hmm. That does seem odd.

Let me try it out.

@puremourning
Copy link
Owner

OK actually I can't. There's another reason we can't move to this version - no macOS binary.

@puremourning
Copy link
Owner

Samsung/netcoredbg#66

@sebaFlame
Copy link
Author

Apparently (Samsung/netcoredbg#65 (comment)) it Just Works™. Using the new https://github.com/Samsung/netcoredbg/releases/download/1.2.0-825/netcoredbg-linux-amd64_fixed.tar.gz @viewizard provided. I seem to be hitting a corner case with the way I'm debugging tests. Exception filters still do not get sent tho.

@puremourning
Copy link
Owner

feel free to use the latest build that works on linux, but I won't be updating vimsepctor while there are known issues and missing platform support. Other clients are also hitting issues : sar/free-omnisharp-vscode-LTS#3

@sebaFlame
Copy link
Author

Fixed and verified by me (Samsung/netcoredbg#65 (comment)). I see no macOS build has been provided yet tho...

@mergify mergify bot closed this as completed in #530 Feb 19, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants