Review Request: konqueror shall allow to click a mailto: link without showing a (...this URL is untrusted...) confirmation dialog
David Faure
faure at kde.org
Thu Jul 12 17:15:53 BST 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105395/#review15734
-----------------------------------------------------------
Ship it!
Looks good. I'm curious as to why this wasn't necessary before, but anyway: please commit this for 4.9, obviously.
- David Faure
On June 30, 2012, 1:21 p.m., Martin Koller wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105395/
> -----------------------------------------------------------
>
> (Updated June 30, 2012, 1:21 p.m.)
>
>
> Review request for kdelibs.
>
>
> Description
> -------
>
> When a non-local webpage contains a mailto: link and you click on it, konqueror shows a confirmation dialog:
>
> This untrusted page links to
> mailto: some.name at some.domain.
> Do you want to follow the link?
>
> (using the page with opera or firefox does not give any such warning and opens kmail)
>
> The check is done by KAuthorized:authorizeUrlAction("redirect", pageURL, linkURL);
>
> This patch fixes this by always allowing a redirection to a mailto link.
>
>
> Diffs
> -----
>
> kdecore/kernel/kauthorized.cpp f7d795a
>
> Diff: http://git.reviewboard.kde.org/r/105395/diff/
>
>
> Testing
> -------
>
> The following simple html page when
> the page is served from a local running HTTP Server
> ( http://localhost:8080/data/test.html )
>
> <html>
> <body>
> <a href="mailto:test.test at test.at">test.test at test.at</a>
> </body>
> </html>
>
>
> Thanks,
>
> Martin Koller
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120712/5f0562e8/attachment.htm>
More information about the kde-core-devel
mailing list