Prevent AccessDeniedException in DEV mode when DEV UI is locating workspace item - #48385
Conversation
michalvavrik
commented
Jun 14, 2025
- closes AccessDeniedException in quarkus dev mode #48310
44ea9a0 to
16ec2da
Compare
| public FileVisitResult preVisitDirectory(Path dir, BasicFileAttributes attrs) throws IOException { | ||
| if (Files.isHidden(dir) || ignoreFolders.contains(dir.getFileName().toString())) { | ||
| if (Files.isHidden(dir) || ignoreFolders.contains(dir.getFileName().toString()) | ||
| || !Files.isReadable(dir)) { |
There was a problem hiding this comment.
For a directory, you also need executable. At least on Unix. Not sure how Windows reacts when checking for executable...
There was a problem hiding this comment.
Thanks, maybe that's why I had to add visitFileFailed (although I'll probably keep it anyway). I'll try it.
There was a problem hiding this comment.
Yes keep it, it’s nice to have a safe guard.
There was a problem hiding this comment.
I have added a check for "executable" file permission and it is in the PR now. But when I tried to reproduce it without visitFileFailed, application still failed to start. So I'll keep it as you are OK with it.
|
Thanks for having a look at this one, I had it on my list and then forgot! |
16ec2da to
02b3476
Compare
This comment has been minimized.
This comment has been minimized.
02b3476 to
3fa48ad
Compare
|
I don't think these failures are related at all, I push the same code again (as it is already rebased on current main) so that we get idea if these failures are stable. |
Status for workflow
|