| Summary: | View Source: Some headers not correctly detected and not bold | ||
|---|---|---|---|
| Product: | [Unmaintained] kmail | Reporter: | hanno |
| Component: | messageviewer | Assignee: | kdepim bugs <pim-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | ruchir.brahmbhatt |
| Priority: | NOR | Keywords: | triaged |
| Version First Reported In: | 1.9.8 | ||
| Target Milestone: | --- | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
| Attachments: | screenshot of undetected header | ||
Created attachment 22161 [details]
screenshot of undetected header
Very similar to bug 88271. The header parsing code isn't very sophisticated there. Reproducible on trunk revision 958061. SVN commit 960620 by jtamate: BUG: 88271 BUG: 152707 Use the right regular expresion to check for mail headers. Also, use a mini ad-hoc state machine to not treat all that looks like headers as headers. When a blank line is found, it is supposed to start a body part, until a header with Content- is found. M +28 -5 mailsourceviewer.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=960620 SVN commit 961651 by tmcguire: Backport r960620 by jtamate from trunk to the 4.2 branch: CCBUG: 88271 CCBUG: 152707 Use the right regular expresion to check for mail headers. Also, use a mini ad-hoc state machine to not treat all that looks like headers as headers. When a blank line is found, it is supposed to start a body part, until a header with Content- is found. M +28 -5 mailsourceviewer.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=961651 |
Version: (using KDE KDE 3.5.8) Installed from: Gentoo Packages Compiler: gcc 4.2.2 OS: Linux When I have an extra-header in a mail that contains a dot, something like: X-foo.bar: test kmail doesn't detect this as a header. According to RFC 822, Section 3.2: field-name = 1*<any CHAR, excluding CTLs, SPACE, and ":"> So a dot should be a perfectly valid char in a header-field. I'll attach an image that shows the header view. It's grey, while field names of detected fields are black.