diff --git a/messageviewer/src/dkim-verify/dkimauthenticationstatusinfo.cpp b/messageviewer/src/dkim-verify/dkimauthenticationstatusinfo.cpp
index 992c16d0..878b0b6f 100644
--- a/messageviewer/src/dkim-verify/dkimauthenticationstatusinfo.cpp
+++ b/messageviewer/src/dkim-verify/dkimauthenticationstatusinfo.cpp
@@ -1,180 +1,195 @@
 /*
    Copyright (C) 2018-2019 Laurent Montel <montel@kde.org>
 
    This library is free software; you can redistribute it and/or
    modify it under the terms of the GNU Library General Public
    License as published by the Free Software Foundation; either
    version 2 of the License, or (at your option) any later version.
 
    This library is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
    Library General Public License for more details.
 
    You should have received a copy of the GNU Library General Public License
    along with this library; see the file COPYING.LIB.  If not, write to
    the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
    Boston, MA 02110-1301, USA.
 */
 
 #include "dkimauthenticationstatusinfo.h"
 #include "dkimauthenticationstatusinfoutil.h"
 #include "messageviewer_dkimcheckerdebug.h"
 
 #include <QRegularExpressionMatch>
 using namespace MessageViewer;
 //see https://tools.ietf.org/html/rfc7601
 DKIMAuthenticationStatusInfo::DKIMAuthenticationStatusInfo()
 {
 }
 
 bool DKIMAuthenticationStatusInfo::parseAuthenticationStatus(const QString &key)
 {
     QString valueKey = key;
     // https://tools.ietf.org/html/rfc7601#section-2.2
     // authres-header = "Authentication-Results:" [CFWS] authserv-id
     //                                            [ CFWS authres-version ]
     //                                            ( no-result / 1*resinfo ) [CFWS] CRLF
 
     // 1) extract AuthservId and AuthVersion
     QRegularExpressionMatch match;
     const QString regStr = DKIMAuthenticationStatusInfoUtil::value_cp() + QLatin1String("(?:") + DKIMAuthenticationStatusInfoUtil::cfws_p() + QLatin1String("([0-9]+)") + DKIMAuthenticationStatusInfoUtil::cfws_op() + QLatin1String(" )?");
     //qDebug() << " regStr" << regStr;
     int index = valueKey.indexOf(QRegularExpression(regStr), 0, &match);
     if (index != -1) {
         mAuthservId = match.captured(1);
         if (!match.captured(2).isEmpty()) {
             mAuthVersion = match.captured(2).toInt();
         } else {
             mAuthVersion = 1;
         }
         valueKey = valueKey.right(valueKey.length() - (index + match.captured(0).length()));
         qDebug() << " match.captured(0)"<<match.captured(0)<<"match.captured(1)" <<match.captured(1) << match.captured(2);
         qDebug() << " valueKey" << valueKey;
     } else {
         return false;
     }
     // check if message authentication was performed
     const QString authResultStr = DKIMAuthenticationStatusInfoUtil::value_cp() + QLatin1String(";") + DKIMAuthenticationStatusInfoUtil::cfws_op() + QLatin1String("?none");
     index = valueKey.indexOf(QRegularExpression(authResultStr), 0, &match);
     if (index != -1) {
         //no result
         return false;
     }
     while (!valueKey.isEmpty()) {
-        mListAuthStatusInfo.append(parseAuthResultInfo(valueKey));
+        const AuthStatusInfo resultInfo = parseAuthResultInfo(valueKey);
+        if (resultInfo.isValid()) {
+            mListAuthStatusInfo.append(resultInfo);
+        }
     }
 
     return true;
 }
 
 DKIMAuthenticationStatusInfo::AuthStatusInfo DKIMAuthenticationStatusInfo::parseAuthResultInfo(QString &valueKey)
 {
     DKIMAuthenticationStatusInfo::AuthStatusInfo authStatusInfo;
     // 2) extract methodspec
     const QString methodVersionp = DKIMAuthenticationStatusInfoUtil::cfws_op() + QLatin1Char('/') + DKIMAuthenticationStatusInfoUtil::cfws_op() + QLatin1String("([0-9]+)");
     const QString method_p = QLatin1Char('(') + DKIMAuthenticationStatusInfoUtil::keyword_p() + QLatin1String(")(?:") + methodVersionp + QLatin1String(")?");
     QString Keyword_result_p = QLatin1String("none|pass|fail|softfail|policy|neutral|temperror|permerror");
     // older SPF specs (e.g. RFC 4408) use mixed case
     Keyword_result_p += QLatin1String("|None|Pass|Fail|SoftFail|Neutral|TempError|PermError");
     const QString result_p = QLatin1Char('=') + DKIMAuthenticationStatusInfoUtil::cfws_op() + QLatin1Char('(') + Keyword_result_p + QLatin1Char(')');
     const QString methodspec_p = QLatin1Char(';') + DKIMAuthenticationStatusInfoUtil::cfws_op() + method_p + DKIMAuthenticationStatusInfoUtil::cfws_op() + result_p;
 
-    qDebug() << "methodspec_p " << methodspec_p;
+    //qDebug() << "methodspec_p " << methodspec_p;
     QRegularExpressionMatch match;
-    const int index = valueKey.indexOf(QRegularExpression(methodspec_p), 0, &match);
+    int index = valueKey.indexOf(QRegularExpression(methodspec_p), 0, &match);
     if (index == -1) {
         valueKey = QString(); //remove it !
-        qDebug() << " not found !!!!!!!!!!!!!!!!!!";
+        qCWarning(MESSAGEVIEWER_DKIMCHECKER_LOG) << "methodspec not found ";
         //no result
         return authStatusInfo;
     }
-    qDebug() << " match" << match.captured(0) << match.captured(1) << match.capturedTexts();
+    //qDebug() << " match" << match.captured(0) << match.captured(1) << match.capturedTexts();
     authStatusInfo.method = match.captured(1);
 
     if (!match.captured(2).isEmpty()) {
         authStatusInfo.methodVersion = match.captured(2).toInt();
     }
     authStatusInfo.result = match.captured(3);
 
+    valueKey = valueKey.right(valueKey.length() - (index + match.captured(0).length())); // Improve it!
     //TODO remove extra text
 
     // 3) extract reasonspec (optional)
     const QString reasonspec_p = QLatin1String("reason") + DKIMAuthenticationStatusInfoUtil::cfws_op() + QLatin1Char('=') + DKIMAuthenticationStatusInfoUtil::cfws_op() + DKIMAuthenticationStatusInfoUtil::value_cp();
+    index = valueKey.indexOf(QRegularExpression(reasonspec_p), 0, &match);
+    if (index != -1) {
+        qDebug() << " reason " << match.capturedTexts();
+    }
+
+
 
 
     // 4) extract propspec (optional)
     const QString pvalue_p = DKIMAuthenticationStatusInfoUtil::value_p() + QLatin1String("|(?:(?:") + DKIMAuthenticationStatusInfoUtil::localPart_p() + QLatin1String("?@)?") + DKIMAuthenticationStatusInfoUtil::domainName_p() + QLatin1Char(')');
 
 
     valueKey = QString(); //remove it !
     return authStatusInfo;
 }
 
 
 int DKIMAuthenticationStatusInfo::authVersion() const
 {
     return mAuthVersion;
 }
 
 void DKIMAuthenticationStatusInfo::setAuthVersion(int authVersion)
 {
     mAuthVersion = authVersion;
 }
 
 QString DKIMAuthenticationStatusInfo::reasonSpec() const
 {
     return mReasonSpec;
 }
 
 void DKIMAuthenticationStatusInfo::setReasonSpec(const QString &reasonSpec)
 {
     mReasonSpec = reasonSpec;
 }
 
 bool DKIMAuthenticationStatusInfo::operator==(const DKIMAuthenticationStatusInfo &other) const
 {
     return mAuthservId == other.authservId()
             && mAuthVersion == other.authVersion()
             && mReasonSpec == other.reasonSpec()
             && mListAuthStatusInfo == other.listAuthStatusInfo();
 }
 
 QList<DKIMAuthenticationStatusInfo::AuthStatusInfo> DKIMAuthenticationStatusInfo::listAuthStatusInfo() const
 {
     return mListAuthStatusInfo;
 }
 
 void DKIMAuthenticationStatusInfo::setListAuthStatusInfo(const QList<AuthStatusInfo> &listAuthStatusInfo)
 {
     mListAuthStatusInfo = listAuthStatusInfo;
 }
 
 QString DKIMAuthenticationStatusInfo::authservId() const
 {
     return mAuthservId;
 }
 
 void DKIMAuthenticationStatusInfo::setAuthservId(const QString &authservId)
 {
     mAuthservId = authservId;
 }
 
 QDebug operator <<(QDebug d, const DKIMAuthenticationStatusInfo &t)
 {
     d << "mAuthservId: " << t.authservId();
     d << "mReasonSpec: " << t.reasonSpec();
     d << "mAuthVersion: " << t.authVersion();
     for (const DKIMAuthenticationStatusInfo::AuthStatusInfo & info : t.listAuthStatusInfo()) {
         d << "mListAuthStatusInfo: " << info.method << " : " << info.result << " : " << info.methodVersion << " : " << info.reason;
     }
     return d;
 }
 
 bool DKIMAuthenticationStatusInfo::AuthStatusInfo::operator==(const DKIMAuthenticationStatusInfo::AuthStatusInfo &other) const
 {
     return other.method == method
             && other.result == result
             && other.methodVersion == methodVersion
             && other.reason == reason;
 }
+
+bool DKIMAuthenticationStatusInfo::AuthStatusInfo::isValid() const
+{
+    return !method.isEmpty();
+}
diff --git a/messageviewer/src/dkim-verify/dkimauthenticationstatusinfo.h b/messageviewer/src/dkim-verify/dkimauthenticationstatusinfo.h
index f6b10f1a..2284930d 100644
--- a/messageviewer/src/dkim-verify/dkimauthenticationstatusinfo.h
+++ b/messageviewer/src/dkim-verify/dkimauthenticationstatusinfo.h
@@ -1,69 +1,70 @@
 /*
    Copyright (C) 2018-2019 Laurent Montel <montel@kde.org>
 
    This library is free software; you can redistribute it and/or
    modify it under the terms of the GNU Library General Public
    License as published by the Free Software Foundation; either
    version 2 of the License, or (at your option) any later version.
 
    This library is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
    Library General Public License for more details.
 
    You should have received a copy of the GNU Library General Public License
    along with this library; see the file COPYING.LIB.  If not, write to
    the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
    Boston, MA 02110-1301, USA.
 */
 
 #ifndef DKIMAUTHENTICATIONSTATUSINFO_H
 #define DKIMAUTHENTICATIONSTATUSINFO_H
 
 #include "messageviewer_private_export.h"
 #include <QString>
 #include <QDebug>
 namespace MessageViewer {
 class MESSAGEVIEWER_TESTS_EXPORT DKIMAuthenticationStatusInfo
 {
 public:
     DKIMAuthenticationStatusInfo();
 
     struct AuthStatusInfo {
         QString method;
         QString result;
         QString reason;
         int methodVersion = -1;
         //TODO add more
         bool operator==(const AuthStatusInfo &other) const;
+        bool isValid() const;
     };
 
     Q_REQUIRED_RESULT QString authservId() const;
     void setAuthservId(const QString &authservId);
 
     Q_REQUIRED_RESULT bool parseAuthenticationStatus(const QString &key);
 
     Q_REQUIRED_RESULT int authVersion() const;
     void setAuthVersion(int authVersion);
 
     Q_REQUIRED_RESULT QString reasonSpec() const;
     void setReasonSpec(const QString &reasonSpec);
 
     Q_REQUIRED_RESULT bool operator==(const DKIMAuthenticationStatusInfo &other) const;
 
     Q_REQUIRED_RESULT QList<AuthStatusInfo> listAuthStatusInfo() const;
     void setListAuthStatusInfo(const QList<AuthStatusInfo> &listAuthStatusInfo);
 
 private:
     AuthStatusInfo parseAuthResultInfo(QString &valueKey);
     QList<AuthStatusInfo> mListAuthStatusInfo;
     QString mAuthservId;
     QString mReasonSpec;
     int mAuthVersion = -1;
 };
 }
 
 Q_DECLARE_METATYPE(MessageViewer::DKIMAuthenticationStatusInfo)
 MESSAGEVIEWER_EXPORT QDebug operator <<(QDebug d, const MessageViewer::DKIMAuthenticationStatusInfo &t);
 
 #endif // DKIMAUTHENTICATIONSTATUSINFO_H