Skip to content

[AGM-2542] Add wfpdiag.cab to crush dump manual #717

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

Merged
merged 9 commits into from
Jul 3, 2025

Conversation

DashaBu
Copy link
Contributor

@DashaBu DashaBu commented Jul 2, 2025

No description provided.

@DashaBu DashaBu requested review from northis and el-termikael July 2, 2025 14:31
Copy link
Contributor

@windsurf-bot windsurf-bot bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other comments (1)
  • docs/guides/minidump.md (63-65) There are a couple of typos in the new instructions:
    1. Click *Start* and type `cmd` to open Command prompt
    
    1. Right-click Command prompt and choose *Run as administrator*
    

💡 To request another review, post a new comment with "/windsurf-review".


1. Reproduce the issue you encountered while using AdGuard

1. Enter the command to stop logging: `netsh wfp capture stop`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider adding a final step that instructs users on how to send the collected wfpdiag.cab file to the support team, since that's the purpose of collecting it.

Copy link

github-actions bot commented Jul 2, 2025

Preview was deployed to: https://pull-request-717.kb-adg.pages.dev/

Co-authored-by: windsurf-bot[bot] <189301087+windsurf-bot[bot]@users.noreply.github.com>

1. Click *Start* and type `cmd` to open Command promt

1. Righ-click Command promp and choose *Run as administrator*
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

repeating numbers 1.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

так должно быть, на сайте будет всё хорошо)


1. Enter the command to start logging: `netsh wfp capture start`

1. Reproduce the issue you encountered while using AdGuard
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Better "...to open a site to pass traffic through AG". It is impossible to repro such bugs that lead to BSOD

Copy link

github-actions bot commented Jul 2, 2025

Preview was deployed to: https://pull-request-717.kb-adg.pages.dev/

Copy link

github-actions bot commented Jul 2, 2025

Preview was deployed to: https://pull-request-717.kb-adg.pages.dev/

Copy link

github-actions bot commented Jul 3, 2025

Preview was deployed to: https://pull-request-717.kb-adg.pages.dev/

4 similar comments
Copy link

github-actions bot commented Jul 3, 2025

Preview was deployed to: https://pull-request-717.kb-adg.pages.dev/

Copy link

github-actions bot commented Jul 3, 2025

Preview was deployed to: https://pull-request-717.kb-adg.pages.dev/

Copy link

github-actions bot commented Jul 3, 2025

Preview was deployed to: https://pull-request-717.kb-adg.pages.dev/

Copy link

github-actions bot commented Jul 3, 2025

Preview was deployed to: https://pull-request-717.kb-adg.pages.dev/

Copy link

github-actions bot commented Jul 3, 2025

Preview was deployed to: https://pull-request-717.kb-adg.pages.dev/

@DashaBu DashaBu merged commit 271b61e into master Jul 3, 2025
4 checks passed
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.

3 participants