Review Request: Add warning for searching the complete file system
Mark Gaiser
markg85 at gmail.com
Thu May 3 20:35:00 BST 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104826/#review13382
-----------------------------------------------------------
Let me first say that i have absolutely nothing to say about this. I do have an opinion about this though.
I would not like this. If i'm searching i just want to search and certainly don't get a dialog just to confirm my search.
Others might think differently though... Perhaps as an option but disabled by default.
- Mark Gaiser
On May 2, 2012, 7:50 p.m., Andre Heinecke wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104826/
> -----------------------------------------------------------
>
> (Updated May 2, 2012, 7:50 p.m.)
>
>
> Review request for KDE Base Apps.
>
>
> Description
> -------
>
> In my company it regularly happens that some "non technical" person, e.g. a secretary or a marketing guy tries to find a file he or she has no idea where it is placed.
> When they use kfind to search the complete file system without using locate this pretty much means a DOS attack on our company network with loads of NFS shares etc.
>
> Searching the complete filesystem is nearly always wrong but as users are used to it from "Search my Computer" or other indexed search options they might not realize it.
>
> This Patch adds a confirmation dialog if you want to search / or file:/// recursively without using locate warning you that this might take a very long time and is a resource intensive operation
>
>
> Diffs
> -----
>
> kfind/kfinddlg.cpp 12b3b7e
> kfind/kquery.h 39609f3
>
> Diff: http://git.reviewboard.kde.org/r/104826/diff/
>
>
> Testing
> -------
>
> Compiled, tested with regular paths, as well as / and file:///
>
>
> Thanks,
>
> Andre Heinecke
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120503/28d59acd/attachment.htm>
More information about the kde-core-devel
mailing list