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

Crashpad disabled regardless of enable-crash-reporter argv #1832

Open
2 tasks done
AgFlore opened this issue Mar 21, 2024 · 0 comments · May be fixed by #1838
Open
2 tasks done

Crashpad disabled regardless of enable-crash-reporter argv #1832

AgFlore opened this issue Mar 21, 2024 · 0 comments · May be fixed by #1838
Labels
bug Something isn't working

Comments

@AgFlore
Copy link

AgFlore commented Mar 21, 2024

Describe the bug
Due to the patch introduced in #787 , regardless of the enable-crash-reporter argv, the crash-reporter is always disabled.

  • unless one start the program with --crash-reporter-directory present.
  • the vscodium patch quits configureCrashReporter if there is no product.appCenter. There always isn't, is it?
  • so there is always no electron crash minidumps for vscodium. Very frustrating for users who come to vscodium just for a pdb-available electron.

Expected behavior

  1. The enable-crash-reporter argv is supposed to be the rightful controller over whether configureCrashReporter should be run. (check https://github.com/microsoft/vscode/blob/8a19756adf1b689e4f490a871d5067e6c54e3d01/src/main.js#L83C54-L83C75 )
  2. If user enable-crash-reporter, the directory set for crashDumps (%appdata/VSCodium/Crashpad by default) should be initilized, containing an empty settings.dat and two directories attachments and reports. Minidumps are to be placed in the reports dir lest a crash happens someday.

My Proposals

  1. The original problem that lead to do not start crashReporter when appCenter isn't configured #787 patch had been resolved by microsoft/vscode@65475dc . Should submitURL be empty or undefined, uploadToServer would be false, making it acceptable to crashReporter.start(). So we might as well remove the crash-reporter.patch.
  2. Document the --crash-reporter-directory <absolute-path> cli option somewhere in the codium repository (based on infomation from vscode), so that users can have a better understanding of the crash telemetry, and quickly get to how to get dumps when electron crashes.
  • From my opinion, this single cli option enables crash dump and disables crash telemetry at the same time, and is very worth recommending to our users.
  • This infomation will also be especially helpful for users who have to stick on an earlier version of codium (permanently affected by the 787-patch!) due to compatibility reasons but also looking for minidumps.
  1. For the current documents, the mentioning of telemetry.enableCrashReporter has been obselete (and potentially misleading!) since the one-time transition in v1.49, and should be updated.

Please confirm that this problem is VSCodium-specific

  • This bug doesn't happen if I use Microsoft's Visual Studio Code. It only happens in VSCodium.

Please confirm that the issue/resolution isn't already documented

@AgFlore AgFlore added the bug Something isn't working label Mar 21, 2024
AgFlore added a commit to AgFlore/vscodium that referenced this issue Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant