From 0c309d6dd8eed9300aff2bd32617028ae77c5dee Mon Sep 17 00:00:00 2001 From: bryan newbold Date: Fri, 12 Apr 2024 14:47:47 -0700 Subject: [PATCH] updates to match sam's output HTML --- bskyweb/cmd/embedr/handlers.go | 13 ++++++++----- bskyweb/cmd/embedr/server.go | 6 +++--- 2 files changed, 11 insertions(+), 8 deletions(-) diff --git a/bskyweb/cmd/embedr/handlers.go b/bskyweb/cmd/embedr/handlers.go index a1cf05b9df..cb287a616a 100644 --- a/bskyweb/cmd/embedr/handlers.go +++ b/bskyweb/cmd/embedr/handlers.go @@ -126,14 +126,16 @@ func (srv *Server) postEmbedHTML(post *appbsky.FeedDefs_PostView) string { if err != nil { log.Error("bad AT-URI in reponse", "aturi", aturi) } - // TODO: could add language, maybe other fiels here? - // XXX: should actually use html/template for this render + // XXX: should use html/template for this render return fmt.Sprintf( - "", - fmt.Sprintf("https://embed.bsky.app/embed/%s/app.bsky.feed.post/%s", post.Author.Did, aturi.RecordKey()), + "

%s

— %s %s
", post.Uri, post.Cid, - "https://embed.bsky.app/iframe-resize.js", + "en", // XXX + "", // XXX + "", // XXX + fmt.Sprintf("%s ", post.IndexedAt), // XXX: Record.CreatedAt + "https://embed.bsky.app/embed.js", // XXX: or iframe-resize.js, for clarity? ) } @@ -143,6 +145,7 @@ func (srv *Server) WebOEmbed(c echo.Context) error { return c.String(http.StatusNotImplemented, "Unsupported oEmbed format: "+formatParam) } + // XXX: do we actually do something with width? width := 550 maxWidthParam := c.QueryParam("maxwidth") if maxWidthParam != "" { diff --git a/bskyweb/cmd/embedr/server.go b/bskyweb/cmd/embedr/server.go index a5398300c4..a5645b361b 100644 --- a/bskyweb/cmd/embedr/server.go +++ b/bskyweb/cmd/embedr/server.go @@ -105,7 +105,7 @@ func serve(cctx *cli.Context) error { e.Use(middleware.LoggerWithConfig(middleware.LoggerConfig{ // Don't log requests for static content. Skipper: func(c echo.Context) bool { - return strings.HasPrefix(c.Request().URL.Path, "/embedr-static") + return strings.HasPrefix(c.Request().URL.Path, "/static") }, })) e.Use(middleware.RateLimiterWithConfig(middleware.RateLimiterConfig{ @@ -139,9 +139,9 @@ func serve(cctx *cli.Context) error { staticHandler := http.FileServer(func() http.FileSystem { if debug { log.Debugf("serving static file from the local file system") - return http.FS(os.DirFS("static")) + return http.FS(os.DirFS("embedr-static")) } - fsys, err := fs.Sub(bskyweb.EmbedrStaticFS, "static") + fsys, err := fs.Sub(bskyweb.EmbedrStaticFS, "embedr-static") if err != nil { log.Fatal(err) }