----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/4927/#review10132 -----------------------------------------------------------
Can you discard this, since it is superceded by http://git.reviewboard.kde.org/r/101241/ ? - Alex On Jan. 21, 2011, 5:37 p.m., José Millán Soto wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/4927/ > ----------------------------------------------------------- > > (Updated Jan. 21, 2011, 5:37 p.m.) > > > Review request for kdelibs. > > > Summary > ------- > > Currently cookies are stored in a plain text file. This patch allows > KCookieJar to store the cookies securely using KWallet. > > The main problem I had writing this patch was that when a web page is > requested, KIO ask for the cookies to kded using dbus. In the first > implementations that I wrote, if the user took too long to open the wallet, > KIO received a dbus timeout. > > To prevent this, if it takes more than 10 seconds to open the wallet, the web > page will be requested without sending the cookies (or sending the available > cookies if there's still the plain text cookie file). If the wallet is opened > after that, the cookies stored in the wallet will be available since then. > > Because of this, the feature is disabled by default. > > > Diffs > ----- > > /trunk/KDE/kdebase/apps/konqueror/settings/kio/kcookiespolicies.cpp 1215885 > /trunk/KDE/kdebase/apps/konqueror/settings/kio/kcookiespoliciesdlg.ui > 1215885 > /trunk/KDE/kdelibs/kioslave/http/kcookiejar/kcookiejar.h 1215938 > /trunk/KDE/kdelibs/kioslave/http/kcookiejar/kcookiejar.cpp 1215938 > /trunk/KDE/kdelibs/kioslave/http/kcookiejar/kcookieserver.h 1215938 > /trunk/KDE/kdelibs/kioslave/http/kcookiejar/kcookieserver.cpp 1215938 > > Diff: http://svn.reviewboard.kde.org/r/4927/diff > > > Testing > ------- > > > Thanks, > > José > >
