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

User running the scan and directory permissions #249

Closed
MaXGitteR opened this issue Jan 14, 2022 · 4 comments
Closed

User running the scan and directory permissions #249

MaXGitteR opened this issue Jan 14, 2022 · 4 comments
Assignees
Labels
discussion question or suggestion enhancement New feature or request patch released

Comments

@MaXGitteR
Copy link

The scan result is not showing the user running the scan.

The head of the result looks like this:

  • Logpresso CVE-2021-44228 Vulnerability Scanner 2.7.1 (2022-01-02)
  • Scanning directory: / (without /dev, /dev/shm, /run, /sys/fs/cgroup, /proc/sys/fs/binfmt_misc, /run/user/30001, /filer, /run/user/30004, /run/user/500)

But it should also mention which user is running the scan.
And it should list the directorys it cant scan because of permissions. During the scan.

@xeraph xeraph self-assigned this Jan 16, 2022
@xeraph xeraph added the discussion question or suggestion label Jan 16, 2022
@xeraph xeraph added enhancement New feature or request patch released labels Feb 2, 2022
@xeraph
Copy link
Contributor

xeraph commented Feb 2, 2022

@MaXGitteR v2.9.0 release reports access denied error for directory traversal failure.
User information will be added in the next release.

@xeraph
Copy link
Contributor

xeraph commented Feb 3, 2022

@MaXGitteR Would you test v2.9.1 release? it print user information like this:

Logpresso CVE-2021-44228 Vulnerability Scanner 2.9.1 (2022-02-03)
Scanning directory by user 'xeraph': d:\tmp2

@MaXGitteR
Copy link
Author

MaXGitteR commented Feb 3, 2022

Hello,
we just updated to 2.9.0. with this version we get the expected messages like this example:
Error: Access denied. Skipping /boot/grub2
this is good for us

2.9.1 - is showing the user now, its also good for us

@xeraph
Copy link
Contributor

xeraph commented Feb 3, 2022

@MaXGitteR Thank you for test report :D

@xeraph xeraph closed this as completed Feb 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion question or suggestion enhancement New feature or request patch released
Projects
None yet
Development

No branches or pull requests

2 participants