summaryrefslogtreecommitdiff
path: root/src/urlbar
diff options
context:
space:
mode:
authorAndrea Diamantini <adjam7@gmail.com>2011-07-11 10:06:41 +0200
committerAndrea Diamantini <adjam7@gmail.com>2011-07-18 11:37:08 +0200
commitc36f3b1351088b86845dd8981a7a9b13b971606e (patch)
treea039935a831f91fccf6e56ffb6e98e08bf27a513 /src/urlbar
parentWARNING: HISTORY_VERSION upgrade!! (diff)
downloadrekonq-c36f3b1351088b86845dd8981a7a9b13b971606e.tar.xz
SSL rewamp, one step to finish!
These are last changes to clean up and improve SSL rekonq communications. It is yet missing a proper SSL errors API integration. You just need to wait for the next commit...
Diffstat (limited to 'src/urlbar')
-rw-r--r--src/urlbar/sslwidget.cpp111
-rw-r--r--src/urlbar/urlbar.cpp5
2 files changed, 86 insertions, 30 deletions
diff --git a/src/urlbar/sslwidget.cpp b/src/urlbar/sslwidget.cpp
index 862bdb3f..8507aa61 100644
--- a/src/urlbar/sslwidget.cpp
+++ b/src/urlbar/sslwidget.cpp
@@ -34,10 +34,11 @@
#include "sslinfodialog.h"
// KDE Includes
+#include <QSslError>
// Qt Includes
#include <QtGui/QDialogButtonBox>
-#include <QtGui/QFormLayout>
+#include <QtGui/QGridLayout>
#include <QtGui/QLabel>
#include <QtGui/QPushButton>
@@ -52,22 +53,28 @@ SSLWidget::SSLWidget(const QUrl &url, const WebSslInfo &info, QWidget *parent)
QSslCertificate cert = info.certificateChain().first();
- QFormLayout *layout = new QFormLayout(this);
+ QGridLayout *layout = new QGridLayout(this);
QLabel *label;
-
+ QLabel *imageLabel;
+
+ int rows = 0;
// ------------------------------------------------------------------------------------------------------------------
+ imageLabel = new QLabel(this);
+ layout->addWidget(imageLabel, rows , 0, Qt::AlignCenter);
+
label = new QLabel(this);
label->setWordWrap(true);
label->setText( i18n("<h4>Identity</h4>") );
- layout->addRow(label);
-
+ layout->addWidget(label, rows++, 1);
+
if (cert.isNull())
{
label = new QLabel(this);
label->setWordWrap(true);
- label->setText( i18n("Warning: this site is carrying a NULL certificate") );
- layout->addRow(label);
+ label->setText( i18n("Warning: this site is carrying a NULL certificate!") );
+
+ imageLabel->setPixmap(KIcon("security-low").pixmap(32));
}
else
{
@@ -75,50 +82,92 @@ SSLWidget::SSLWidget(const QUrl &url, const WebSslInfo &info, QWidget *parent)
{
label = new QLabel(this);
label->setWordWrap(true);
- label->setText( i18n("This certificate for this site is Valid and has been verified by ")
- + cert.issuerInfo(QSslCertificate::CommonName) );
- layout->addRow(label);
+ label->setText( i18n("This certificate for this site is valid and has been verified by:\n%1.",
+ cert.issuerInfo(QSslCertificate::CommonName)) );
+
+
+ imageLabel->setPixmap(KIcon("security-high").pixmap(32));
}
else
{
+ QString errors;
+ QStringList sl = m_info.certificateErrors().split("\t", QString::SkipEmptyParts);
+ Q_FOREACH(const QString &s, sl)
+ {
+ bool didConvert;
+ QSslError::SslError error = static_cast<QSslError::SslError>(s.trimmed().toInt(&didConvert));
+ if (didConvert)
+ {
+ errors += QSslError(error).errorString() + QL1S("\n");
+ }
+ }
label = new QLabel(this);
label->setWordWrap(true);
- label->setText( i18n("Warning: The certificate for this site is NOT valid!") );
- layout->addRow(label);
+ label->setText( i18n("The certificate for this site is NOT valid for the following reasons:\n%1", errors) );
+
+ imageLabel->setPixmap(KIcon("security-medium").pixmap(32));
}
}
-
+
+ layout->addWidget(label, rows++, 1);
+
label = new QLabel(this);
label->setWordWrap(true);
label->setText("<a href=\"moresslinfos\">Certificate Information</a>");
connect(label, SIGNAL(linkActivated(const QString &)), this, SLOT(showMoreSslInfos(const QString &)));
- layout->addRow(label);
+ layout->addWidget(label, rows++, 1);
// ------------------------------------------------------------------------------------------------------------------
label = new QLabel(this);
label->setWordWrap(true);
- label->setText( QL1S("<hr /><h4>Encryption</h4>") ); // ----------------------------------------------- //
- layout->addRow(label);
+ label->setText( QL1S("<h4>Encryption</h4>") ); // ----------------------------------------------- //
+ layout->addWidget(label, rows, 1);
+
+ imageLabel = new QLabel(this);
+ layout->addWidget(imageLabel, rows++ , 0, Qt::AlignCenter);
if (cert.isNull())
{
label = new QLabel(this);
label->setWordWrap(true);
label->setText( i18n("Your connection to %1 is NOT encrypted!!\n\n", m_url.host()) );
- layout->addRow(label);
+ layout->addWidget(label, rows++ , 1);
+
+ imageLabel->setPixmap(KIcon("security-low").pixmap(32));
}
else
{
label = new QLabel(this);
label->setWordWrap(true);
- label->setText( i18n("Your connection to %1 is encrypted with %2 encryption\n\n", m_url.host(), m_info.supportedChiperBits()) );
- layout->addRow(label);
+ label->setText( i18n("Your connection to %1 is encrypted with %2-bit encryption.\n\n", m_url.host(), m_info.supportedChiperBits()) );
+ layout->addWidget(label, rows++, 1);
- QString sslVersion = QL1S("SSLv") + cert.version();
+ int vers = cert.version().toInt();
+ QString sslVersion;
+ switch(vers)
+ {
+ case 0:
+ sslVersion = QL1S("SSL 3.0");
+ imageLabel->setPixmap(KIcon("security-high").pixmap(32));
+ break;
+ case 1:
+ sslVersion = QL1S("SSL 2.0");
+ imageLabel->setPixmap(KIcon("security-medium").pixmap(32));
+ break;
+ case 2:
+ case 3:
+ sslVersion = QL1S("TLS 1.0");
+ imageLabel->setPixmap(KIcon("security-high").pixmap(32));
+ break;
+ default:
+ sslVersion = QL1S("Unknown");
+ imageLabel->setPixmap(KIcon("security-medium").pixmap(32));
+ }
+
label = new QLabel(this);
label->setWordWrap(true);
- label->setText( i18n("The connection uses %1\n\n", sslVersion) );
- layout->addRow(label);
+ label->setText( i18n("The connection uses %1.\n\n", sslVersion) );
+ layout->addWidget(label, rows++, 1);
const QStringList cipherInfo = m_info.ciphers().split('\n', QString::SkipEmptyParts);
label = new QLabel(this);
@@ -129,14 +178,18 @@ SSLWidget::SSLWidget(const QUrl &url, const WebSslInfo &info, QWidget *parent)
m_info.usedChiperBits(),
cipherInfo[3],
cipherInfo[1]) );
- layout->addRow(label);
+ layout->addWidget(label, rows++, 1);
+
}
// ------------------------------------------------------------------------------------------------------------------
+ imageLabel = new QLabel(this);
+ layout->addWidget(imageLabel, rows , 0, Qt::AlignCenter);
+
label = new QLabel(this);
label->setWordWrap(true);
- label->setText( i18n("<hr /><h4>Site Information</h4>") );
- layout->addRow(label);
+ label->setText( i18n("<h4>Site Information</h4>") );
+ layout->addWidget(label, rows++, 1);
label = new QLabel(this);
label->setWordWrap(true);
@@ -145,13 +198,15 @@ SSLWidget::SSLWidget(const QUrl &url, const WebSslInfo &info, QWidget *parent)
if (firstVisit.visitCount == 1)
{
- label->setText( i18n("It's your first time visiting this site") );
+ label->setText( i18n("It's your first time visiting this site!") );
+ imageLabel->setPixmap(KIcon("security-medium").pixmap(32));
}
else
{
- label->setText( i18n("You just visited this site!\nYour first visit was on %1", firstVisit.firstDateTimeVisit.toString()) );
+ label->setText( i18n("You just visited this site!\nYour first visit was on %1.\n", firstVisit.firstDateTimeVisit.toString()) );
+ imageLabel->setPixmap(KIcon("security-high").pixmap(32));
}
- layout->addRow(label);
+ layout->addWidget(label, rows++, 1);
// -----------------------------------------------------------------------------------
setLayout(layout);
diff --git a/src/urlbar/urlbar.cpp b/src/urlbar/urlbar.cpp
index fa23b6c6..8b3dac65 100644
--- a/src/urlbar/urlbar.cpp
+++ b/src/urlbar/urlbar.cpp
@@ -356,6 +356,7 @@ void UrlBar::loadFinished()
// show SSL
if(_tab->url().scheme() == QL1S("https"))
{
+ // NOTE: the choice for the right SSL icon is done in the addRightIcon method
IconButton *bt = addRightIcon(UrlBar::SSL);
connect(bt, SIGNAL(clicked(QPoint)), _tab->page(), SLOT(showSSLInfo(QPoint)));
}
@@ -465,8 +466,8 @@ IconButton *UrlBar::addRightIcon(UrlBar::icon ic)
break;
case UrlBar::SSL:
_tab->page()->hasSslValid()
- ? rightIcon->setIcon(KIcon("security-high"))
- : rightIcon->setIcon(KIcon("security-low"));
+ ? rightIcon->setIcon(KIcon("object-locked"))
+ : rightIcon->setIcon(KIcon("object-unlocked"));
rightIcon->setToolTip(i18n("Show SSL Info"));
break;
case UrlBar::BK: