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.

Sign up for GitHub

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

Jump to bottom

Check for Administrator access on Windows #451

Closed
fujiapple852 opened this issue Mar 25, 2023 · 2 comments · Fixed by #463
Closed

Check for Administrator access on Windows #451

fujiapple852 opened this issue Mar 25, 2023 · 2 comments · Fixed by #463
Labels
enhancement New feature or request windows
Milestone
0.8.0

Comments

@fujiapple852
Copy link
Owner

Trippy requires Administrator access on Windows (for raw sockets) but, unlike the other platforms, does not currently validate that the user has the required access on startup.
@fujiapple852 fujiapple852 added this to the 0.8.0 milestone Mar 25, 2023
@fujiapple852 fujiapple852 added the enhancement New feature or request label Mar 25, 2023
@fujiapple852
Copy link
Owner Author

@zarkdav do you know how to check if a user is an Administrator on Windows?

@fujiapple852 fujiapple852 mentioned this issue Mar 26, 2023
Merged
@fujiapple852
Copy link
Owner Author

fujiapple852 commented Mar 26, 2023 •
edited

Added a modified version of this rust code which is derived from this c++ code in #463 and it seems to work in a Windows VM.

@fujiapple852 fujiapple852 self-assigned this Mar 26, 2023
fujiapple852 added a commit that referenced this issue Apr 13, 2023
@fujiapple852
doc: add #451 to CHANGELOG.md
7f927cd
fujiapple852 added a commit that referenced this issue Apr 13, 2023
@fujiapple852
doc: add #451 to CHANGELOG.md
2f43f75
@fujiapple852 fujiapple852 removed their assignment Apr 15, 2023
@p-linnane p-linnane mentioned this issue May 15, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request windows
Projects
None yet
Milestone
0.8.0
Development

Successfully merging a pull request may close this issue.

Ensure administrator privileges on Windows fujiapple852/trippy
1 participant
@fujiapple852