Merge lp:~sergiodj/midori/sdj-ssl-error-improve into lp:midori

Proposed by Sergio Durigan Junior
Status: Merged
Approved by: Cris Dywan
Approved revision: 7042
Merged at revision: 7042
Proposed branch: lp:~sergiodj/midori/sdj-ssl-error-improve
Merge into: lp:midori
Diff against target: 15 lines (+4/-1)
1 file modified
midori/midori-view.c (+4/-1)
To merge this branch: bzr merge lp:~sergiodj/midori/sdj-ssl-error-improve
Reviewer Review Type Date Requested Status
Cris Dywan Approve
Review via email: mp+272525@code.launchpad.net

Commit message

Improve error message when "Trust this website" doesn't work

Description of the change

Improve error message when gnome-keyring cannot be found, Midori is accessing a website with a self-signed SSL/TLS certificate, and the user clicks on "Trust this website".

To post a comment you must log in.
Revision history for this message
Cris Dywan (kalikiana) wrote :

Reviewing this made me realize it's annoying to test since there's no trivial way to get Midori to pretend that the keyring is absent... at least something to consider for the future.

Looks very nice indeed!
Note for next time, always set a commit message in Launchpad, otherwise the merge bot will not touch it until you do.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'midori/midori-view.c'
2--- midori/midori-view.c 2015-09-25 21:30:31 +0000
3+++ midori/midori-view.c 2015-09-26 22:40:41 +0000
4@@ -623,7 +623,10 @@
5 gchar* slots = g_strjoinv (" , ", (gchar**)gcr_pkcs11_get_trust_lookup_uris ());
6 gchar* title = g_strdup_printf ("Error granting trust: %s", error->message);
7 midori_tab_stop_loading (MIDORI_TAB (view));
8- midori_view_display_error (view, NULL, NULL, NULL, title, slots, NULL,
9+ midori_view_display_error (view, NULL, NULL, NULL, title, slots, _("Please look at our "
10+ "<a href=\"http://midori-browser.org/faqs/\" target=\"_blank\">FAQ</a>, section "
11+ "\"<a href=\"http://midori-browser.org/faqs/#security_features\" target=\"_blank\">"
12+ "Security Features</a>\", to understand how you can solve this problem."),
13 _("Trust this website"), NULL);
14 g_free (title);
15 g_free (slots);

Subscribers

People subscribed via source and target branches