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

Add Command Line Arguments to DefaultContext #697

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jayhf
Copy link

@jayhf jayhf commented Nov 22, 2024

The command line arguments seem like a really useful piece of information to have when debugging crashes. This adds them to the DefaultContext so that they're readily available in Sentry

@jayhf jayhf requested a review from tustanivsky as a code owner November 22, 2024 17:11
@tustanivsky
Copy link
Collaborator

Hey @jayhf, I believe we're already adding the command line input to Crash Info context on Win/Linux for captured crash events:

ContextValues.Add("Command Line", FString(SessionContext.CommandLine).TrimStart());

Does it miss some bits of info you expect to be there along with other CL arguments?

@jayhf
Copy link
Author

jayhf commented Nov 26, 2024

Ah, that's great to know. I didn't realize. Where exactly can those context be found? I haven't seen any of those fields in the crash logs I'm analyzing. Are there any docs you'd recommend for what information is available where?

@tustanivsky
Copy link
Collaborator

The command line arguments can be found at captured crash event's page in Contexts -> Crash Info section in case you're using Sentry's plugin integration for error handling:

image

If you're sending crashes to Sentry with the default Crash Reporter instead then CL args should be listed in Contexts -> unreal -> custom.

Also, you can consider enriching the events with custom contexts via the plugin's API if the default one misses some bits of info you're interested in.

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

Successfully merging this pull request may close these issues.

2 participants