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

Does it require admin permissions for it to work on win7 ? #2

Closed
minanagehsalalma opened this issue Mar 3, 2020 · 21 comments
Closed
Assignees

Comments

@minanagehsalalma
Copy link

No description provided.

@warengonzaga
Copy link
Owner

Hi @minanagehsalalma yes it requires permissions to work

@minanagehsalalma
Copy link
Author

Hi @minanagehsalalma yes it requires permissions to work

@warengonzaga but why ? It works on win 8 and 10 without it ....
Is this for all commands or just the export or command ?
Does doing it one network by one manually fixes it ?

@warengonzaga
Copy link
Owner

@minanagehsalalma it works well in Windows 8 and 10 but I'm not sure in Windows 7... I will try it on my VM if it will work... I guess there is a command in the program that is unknown to Windows 7.

@warengonzaga warengonzaga self-assigned this Mar 3, 2020
@minanagehsalalma
Copy link
Author

minanagehsalalma commented Mar 3, 2020 via email

@warengonzaga
Copy link
Owner

warengonzaga commented Mar 3, 2020

@minanagehsalalma I found the issue working on a fix...

@minanagehsalalma
Copy link
Author

minanagehsalalma commented Mar 3, 2020 via email

@warengonzaga
Copy link
Owner

@minanagehsalalma the way how it checks for admin privileges...

@minanagehsalalma
Copy link
Author

minanagehsalalma commented Mar 3, 2020 via email

@warengonzaga
Copy link
Owner

@minanagehsalalma it should be run in admin state... since some part of the program needs an admin privilege... will get back to you once fix is available.

@minanagehsalalma
Copy link
Author

minanagehsalalma commented Mar 3, 2020 via email

@warengonzaga
Copy link
Owner

warengonzaga commented Mar 3, 2020

@minanagehsalalma in your copy open the .bat file and remove lines 36 to 42 and 48 to 66 and run it as admin. It should work... but I'm working on a fix so you don't have to do this in the next version.

@minanagehsalalma
Copy link
Author

minanagehsalalma commented Mar 3, 2020 via email

@warengonzaga
Copy link
Owner

@minanagehsalalma nope... it was a typo... "and run it as admin."

@minanagehsalalma
Copy link
Author

minanagehsalalma commented Mar 3, 2020 via email

@warengonzaga
Copy link
Owner

Some part of it the program requires admin privileges... I'm currently working on it... and if admin privileges not really needed I will remove it.

@rewind99
Copy link

rewind99 commented Mar 3, 2020

This is not a Tool for Users instead its for Admins - for me its totally ok to do it with admin rights It would be nice to enter my credentials anywhere in the script so i didnt have to type em in.

@warengonzaga
Copy link
Owner

@rewind99 thanks... but as per checking... the program still works even without admin rights... so the admin checking part is kind of useless... I might remove it... and change the program for users. Let me work on the fix...

@rewind99
Copy link

rewind99 commented Mar 3, 2020

At least its always possible to right click and run as admin...

@warengonzaga warengonzaga added resolved and removed bug labels Mar 3, 2020
@warengonzaga
Copy link
Owner

@rewind99 and @minanagehsalalma this issue has been fixed! Check release 2.5.0
https://github.com/WarenGonzaga/wifi-passview/releases/tag/2.5.0

Let me know if you need any help... I'm closing this now...

@minanagehsalalma
Copy link
Author

Some part of it the program requires admin privileges.

@warengonzaga which part exactly that requires that all the options in netsh works without admin privileges.

@rewind99 why do you think this a tool for admins ?
It should be for everyone not just admins !

@minanagehsalalma
Copy link
Author

@warengonzaga thanks for the update mate ;) really appreciate it

Repository owner locked as resolved and limited conversation to collaborators Mar 5, 2020
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.

3 participants