The forward quote header renders "From: Name <email>", but the HTML variant interpolated the sender string unescaped. In the rich-text composer the "<email>" portion is parsed by the browser as a bogus HTML tag and dropped, so the address silently disappears - the user sees only "From: Display Name". The plain-text variant and the details panel escape correctly, which is why the address shows there. This is the regression from #367, which added the "<email>" into the HTML string without escaping it. Fix: HTML-escape the user-controlled values (sender, subject, date) in every HTML quote-header path - the production builder in lib/quote-header.ts and the composer's inline fallback (both htmlBody and plain-body branches), for forward and reply. The reply line keeps the bare display name by design (#367), but its HTML form is now escaped too so a display name containing markup can't break out. As a side benefit this closes an HTML-injection vector: a crafted subject or display name was previously injected raw into the composer document. Adds lib/__tests__/quote-header.test.ts covering: forward text keeps "Name <email>"; forward HTML escapes the angle brackets (address survives) and a markup subject/display name; reply stays bare-name and HTML-safe.
93 lines
3.4 KiB
TypeScript
93 lines
3.4 KiB
TypeScript
import { describe, it, expect } from 'vitest';
|
|
import { buildQuoteHeader } from '@/lib/quote-header';
|
|
|
|
const base = {
|
|
newTo: [] as string[],
|
|
newCc: [] as string[],
|
|
locale: 'en',
|
|
timeFormat: '24h' as const,
|
|
unknownLabel: 'Unknown',
|
|
};
|
|
|
|
const sender = { name: 'Display Name', email: 'user@domain.tld' };
|
|
|
|
describe('buildQuoteHeader (#482 — sender address survives HTML rendering)', () => {
|
|
it('forward TEXT keeps the full "Name <email>" sender', async () => {
|
|
const h = await buildQuoteHeader({
|
|
mode: 'forward',
|
|
email: { from: [sender], subject: 'Hello', receivedAt: '2026-01-01T10:00:00Z' },
|
|
...base,
|
|
});
|
|
expect(h.text).toContain('From: Display Name <user@domain.tld>');
|
|
});
|
|
|
|
it('forward HTML escapes the angle brackets so the address is not eaten as a tag', async () => {
|
|
const h = await buildQuoteHeader({
|
|
mode: 'forward',
|
|
email: { from: [sender], subject: 'Hello', receivedAt: '2026-01-01T10:00:00Z' },
|
|
...base,
|
|
});
|
|
// The regression: a raw "<user@domain.tld>" is parsed as an HTML tag by the
|
|
// rich-text composer and dropped, leaving only "From: Display Name".
|
|
expect(h.html).toContain('Display Name <user@domain.tld>');
|
|
expect(h.html).not.toContain('<user@domain.tld>');
|
|
});
|
|
|
|
it('forward HTML escapes a subject containing markup (injection hardening)', async () => {
|
|
const h = await buildQuoteHeader({
|
|
mode: 'forward',
|
|
email: { from: [sender], subject: 'Hi <b>x</b>', receivedAt: '2026-01-01T10:00:00Z' },
|
|
...base,
|
|
});
|
|
expect(h.html).toContain('Hi <b>x</b>');
|
|
expect(h.html).not.toContain('<b>x</b>');
|
|
});
|
|
|
|
it('forward HTML escapes a malicious display name', async () => {
|
|
const h = await buildQuoteHeader({
|
|
mode: 'forward',
|
|
email: {
|
|
from: [{ name: '<img src=x onerror=alert(1)>', email: 'evil@x.tld' }],
|
|
subject: 'Hello',
|
|
receivedAt: '2026-01-01T10:00:00Z',
|
|
},
|
|
...base,
|
|
});
|
|
expect(h.html).not.toContain('<img src=x');
|
|
expect(h.html).toContain('<img src=x');
|
|
});
|
|
|
|
it('reply line includes the full "Name <email>" sender, escaped in HTML', async () => {
|
|
const h = await buildQuoteHeader({
|
|
mode: 'reply',
|
|
email: { from: [sender], subject: 'Hello', receivedAt: '2026-01-01T10:00:00Z' },
|
|
...base,
|
|
});
|
|
// TEXT keeps the real angle brackets ("On <date>, Display Name <user@domain.tld> wrote:").
|
|
expect(h.text).toContain('Display Name <user@domain.tld> wrote:');
|
|
// HTML escapes them so the address survives the rich-text editor (#482).
|
|
expect(h.html).toContain('Display Name <user@domain.tld>');
|
|
expect(h.html).not.toContain('<user@domain.tld>');
|
|
});
|
|
|
|
it('reply line stays HTML-safe for a display name containing markup', async () => {
|
|
const evil = await buildQuoteHeader({
|
|
mode: 'reply',
|
|
email: { from: [{ name: '<b>x</b>', email: 'e@x.tld' }], subject: 'Hello', receivedAt: '2026-01-01T10:00:00Z' },
|
|
...base,
|
|
});
|
|
expect(evil.html).not.toContain('<b>x</b>');
|
|
expect(evil.html).toContain('<b>x</b>');
|
|
});
|
|
|
|
it('reply line falls back to bare email when there is no display name', async () => {
|
|
const h = await buildQuoteHeader({
|
|
mode: 'reply',
|
|
email: { from: [{ email: 'noname@x.tld' }], subject: 'Hello', receivedAt: '2026-01-01T10:00:00Z' },
|
|
...base,
|
|
});
|
|
expect(h.text).toContain('noname@x.tld wrote:');
|
|
expect(h.text).not.toContain('<noname@x.tld>');
|
|
});
|
|
});
|