web/notify.moe
Code Issues Pull Requests Activity

Deterministic output

Browse Source
This commit is contained in:
Eduard Urbach 2017-11-09 08:45:51 +01:00
parent 423ad01af2
commit 454787b52d
2 changed files with 31 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
layout/layout.go
View File

@ -1,6 +1,8 @@
package layout
import (
"sort"
"github.com/aerogo/aero"
"github.com/animenotifier/arn"
"github.com/animenotifier/notify.moe/components"
@ -11,5 +13,25 @@ import (
func Render(ctx *aero.Context, content string) string {
user := utils.GetUser(ctx)
openGraph, _ := ctx.Data.(*arn.OpenGraph)
return components.Layout(ctx.App, ctx, user, openGraph, content)
// Make output order deterministic to profit from Aero caching.
// To do this, we need to create slices and sort the tags.
var meta []string
var tags []string
if openGraph != nil {
for name := range openGraph.Meta {
meta = append(meta, name)
}
sort.Strings(meta)
for name := range openGraph.Tags {
tags = append(tags, name)
}
sort.Strings(tags)
}
return components.Layout(ctx.App, ctx, user, openGraph, meta, tags, content)
}

16
layout/layout.pixy
View File

@ -1,4 +1,4 @@
component Layout(app *aero.Application, ctx *aero.Context, user *arn.User, openGraph *arn.OpenGraph, content string)
component Layout(app *aero.Application, ctx *aero.Context, user *arn.User, openGraph *arn.OpenGraph, meta, tags []string, content string)
html(lang="en")
head
if openGraph != nil
@ -8,14 +8,14 @@ component Layout(app *aero.Application, ctx *aero.Context, user *arn.User, openG
meta(name="viewport", content="width=device-width, minimum-scale=1.0, initial-scale=1.0, user-scalable=yes")
meta(name="theme-color", content=app.Config.Manifest.ThemeColor)
if openGraph != nil
for name, value := range openGraph.Meta
meta(name=name, content=value)
for property, content := range openGraph.Tags
meta(property=property, content=content)
for _, name := range meta
meta(name=name, content=openGraph.Meta[name])
for _, name := range tags
meta(property=name, content=openGraph.Tags[name])
link(rel="chrome-webstore-item", href="https://chrome.google.com/webstore/detail/hajchfikckiofgilinkpifobdbiajfch")
link(rel="manifest", href="/manifest.json")
body