Turn message part viewers into plugins

This commit is contained in:
Simon Ser 2020-02-12 14:42:51 +01:00
parent 892f1fa581
commit 8299617ebc
No known key found for this signature in database
GPG key ID: 0FDE7BE0E88F5E48
15 changed files with 178 additions and 40 deletions

2
.gitignore vendored
View file

@ -6,3 +6,5 @@
!/plugins/caldav
!/plugins/carddav
!/plugins/lua
!/plugins/viewhtml
!/plugins/viewtext

View file

@ -16,6 +16,8 @@ import (
_ "git.sr.ht/~emersion/koushin/plugins/caldav"
_ "git.sr.ht/~emersion/koushin/plugins/carddav"
_ "git.sr.ht/~emersion/koushin/plugins/lua"
_ "git.sr.ht/~emersion/koushin/plugins/viewhtml"
_ "git.sr.ht/~emersion/koushin/plugins/viewtext"
)
func main() {

View file

@ -1,3 +1,2 @@
<script src="/plugins/base/assets/script.js"></script>
</body>
</html>

View file

@ -3,6 +3,5 @@
<head>
<meta charset="utf-8">
<title>koushin</title>
<link rel="stylesheet" href="/plugins/base/assets/style.css">
</head>
<body>

View file

@ -110,7 +110,7 @@
<hr>
{{if .Body}}
{{if .View}}
<p>
{{if .Message.HasFlag "\\Draft"}}
<a href="{{.Message.Uid}}/edit?part={{.PartPath}}">Edit draft</a>
@ -118,13 +118,7 @@
<a href="{{.Message.Uid}}/reply?part={{.PartPath}}">Reply</a>
{{end}}
</p>
{{if .IsHTML}}
<!-- allow-same-origin is required to resize the frame with its content -->
<!-- allow-popups is required for target="_blank" links -->
<iframe id="email-frame" srcdoc="{{.Body}}" sandbox="allow-same-origin allow-popups"></iframe>
{{else}}
<pre>{{.Body}}</pre>
{{end}}
{{.View}}
{{else}}
<p>Can't preview this message part.</p>
<a href="{{.Message.Uid}}/raw?part={{.PartPath}}">Download</a>

View file

@ -176,8 +176,7 @@ type MessageRenderData struct {
Mailboxes []*imap.MailboxInfo
Mailbox *imap.MailboxStatus
Message *IMAPMessage
Body string
IsHTML bool
View interface{}
PartPath string
MailboxPage int
Flags map[string]bool
@ -255,21 +254,9 @@ func handleGetPart(ctx *koushin.Context, raw bool) error {
}
}
var body []byte
if strings.HasPrefix(strings.ToLower(mimeType), "text/") {
body, err = ioutil.ReadAll(part.Body)
if err != nil {
return fmt.Errorf("failed to read part body: %v", err)
}
}
isHTML := false
if strings.EqualFold(mimeType, "text/html") {
body, err = sanitizeHTML(body)
if err != nil {
return fmt.Errorf("failed to sanitize HTML part: %v", err)
}
isHTML = true
view, err := viewMessagePart(ctx, msg, part)
if err == ErrViewUnsupported {
view = nil
}
flags := make(map[string]bool)
@ -286,8 +273,7 @@ func handleGetPart(ctx *koushin.Context, raw bool) error {
Mailboxes: mailboxes,
Mailbox: mbox,
Message: msg,
Body: string(body),
IsHTML: isHTML,
View: view,
PartPath: partPathString,
MailboxPage: int(mbox.Messages-msg.SeqNum) / messagesPerPage,
Flags: flags,

38
plugins/base/viewer.go Normal file
View file

@ -0,0 +1,38 @@
package koushinbase
import (
"fmt"
"git.sr.ht/~emersion/koushin"
"github.com/emersion/go-message"
)
// ErrViewUnsupported is returned by Viewer.ViewMessagePart when the message
// part isn't supported.
var ErrViewUnsupported = fmt.Errorf("cannot generate message view: unsupported part")
// Viewer is a message part viewer.
type Viewer interface {
// ViewMessagePart renders a message part. The returned value is displayed
// in a template. ErrViewUnsupported is returned if the message part isn't
// supported.
ViewMessagePart(*koushin.Context, *IMAPMessage, *message.Entity) (interface{}, error)
}
var viewers []Viewer
// RegisterViewer registers a message part viewer.
func RegisterViewer(viewer Viewer) {
viewers = append(viewers, viewer)
}
func viewMessagePart(ctx *koushin.Context, msg *IMAPMessage, part *message.Entity) (interface{}, error) {
for _, viewer := range viewers {
v, err := viewer.ViewMessagePart(ctx, msg, part)
if err == ErrViewUnsupported {
continue
}
return v, err
}
return nil, ErrViewUnsupported
}

View file

@ -0,0 +1,10 @@
package koushinviewhtml
import (
"git.sr.ht/~emersion/koushin"
)
func init() {
p := koushin.GoPlugin{Name: "viewhtml"}
koushin.RegisterPluginLoader(p.Loader())
}

View file

@ -1,4 +1,4 @@
package koushinbase
package koushinviewhtml
import (
"bytes"

View file

@ -0,0 +1,57 @@
package koushinviewhtml
import (
"bytes"
"fmt"
"html/template"
"io/ioutil"
"strings"
"git.sr.ht/~emersion/koushin"
koushinbase "git.sr.ht/~emersion/koushin/plugins/base"
"github.com/emersion/go-message"
)
const tpl = `
<!-- allow-same-origin is required to resize the frame with its content -->
<!-- allow-popups is required for target="_blank" links -->
<iframe id="email-frame" srcdoc="{{.}}" sandbox="allow-same-origin allow-popups"></iframe>
<script src="/plugins/viewhtml/assets/script.js"></script>
<link rel="stylesheet" href="/plugins/viewhtml/assets/style.css">
`
type viewer struct{}
func (viewer) ViewMessagePart(ctx *koushin.Context, msg *koushinbase.IMAPMessage, part *message.Entity) (interface{}, error) {
mimeType, _, err := part.Header.ContentType()
if err != nil {
return nil, err
}
if !strings.EqualFold(mimeType, "text/html") {
return nil, koushinbase.ErrViewUnsupported
}
body, err := ioutil.ReadAll(part.Body)
if err != nil {
return nil, fmt.Errorf("failed to read part body: %v", err)
}
body, err = sanitizeHTML(body)
if err != nil {
return nil, fmt.Errorf("failed to sanitize HTML part: %v", err)
}
t := template.Must(template.New("view-html.html").Parse(tpl))
var buf bytes.Buffer
err = t.Execute(&buf, string(body))
if err != nil {
return nil, err
}
return template.HTML(buf.String()), nil
}
func init() {
koushinbase.RegisterViewer(viewer{})
}

View file

@ -0,0 +1,10 @@
package koushinviewtext
import (
"git.sr.ht/~emersion/koushin"
)
func init() {
p := koushin.GoPlugin{Name: "viewtext"}
koushin.RegisterPluginLoader(p.Loader())
}

View file

@ -0,0 +1,49 @@
package koushinviewtext
import (
"bytes"
"fmt"
"html/template"
"io/ioutil"
"strings"
"git.sr.ht/~emersion/koushin"
koushinbase "git.sr.ht/~emersion/koushin/plugins/base"
"github.com/emersion/go-message"
)
// TODO: dim quotes and "On xxx, xxx wrote:" lines
// TODO: turn URLs into links
const tpl = `<pre>{{.}}</pre>`
type viewer struct{}
func (viewer) ViewMessagePart(ctx *koushin.Context, msg *koushinbase.IMAPMessage, part *message.Entity) (interface{}, error) {
mimeType, _, err := part.Header.ContentType()
if err != nil {
return nil, err
}
if !strings.EqualFold(mimeType, "text/plain") {
return nil, koushinbase.ErrViewUnsupported
}
body, err := ioutil.ReadAll(part.Body)
if err != nil {
return nil, fmt.Errorf("failed to read part body: %v", err)
}
t := template.Must(template.New("view-text.html").Parse(tpl))
var buf bytes.Buffer
err = t.Execute(&buf, string(body))
if err != nil {
return nil, err
}
return template.HTML(buf.String()), nil
}
func init() {
koushinbase.RegisterViewer(viewer{})
}

View file

@ -99,16 +99,8 @@
{{end}}
</ul>
{{if .Body}}
{{if .IsHTML}}
<!-- allow-same-origin is required to resize the frame with its content -->
<!-- allow-popups is required for target="_blank" links -->
<iframe id="email-frame"
srcdoc="{{.Body}}"
sandbox="allow-same-origin allow-popups"></iframe>
{{else}}
<pre>{{.Body}}</pre>
{{end}}
{{if .View}}
{{.View}}
{{else}}
<p>Can't preview this message part.</p>
<a href="{{.Message.Uid}}/raw?part={{.PartPath}}">Download</a>