Dockerfile

@@ -25,7 +25,6 @@ FROM alpine:latest
 
 RUN apk --no-cache add ca-certificates
 RUN mkdir -p /data
-RUN mkdir -p /scratch
 
 WORKDIR /root/
 
@@ -35,7 +34,6 @@ COPY --from=builder /app/static ./static
 COPY --from=builder /app/views ./views
 
 ENV DATA_DIR=/data
-ENV SCRATCH_DIR=/scratch
 
 EXPOSE 3000
 

cmds/server.go

@@ -116,7 +116,7 @@ Starting weiro without any arguments will start the server.
 			app.Post("/login", lh.DoLogin)
 			app.Post("/logout", lh.Logout)
 
-			siteGroup := app.Group("/sites/:siteID", middleware.LogErrors(), middleware.RequireUser(svcs.Auth), middleware.RequiresSite(svcs.Sites))
+			siteGroup := app.Group("/sites/:siteID", middleware.RequireUser(svcs.Auth), middleware.RequiresSite(svcs.Sites))
 
 			siteGroup.Get("/posts", ph.Index)
 			siteGroup.Get("/posts/new", ph.New)

config/config.go

@@ -9,7 +9,7 @@ import (
 
 type Config struct {
 	DataDir     string `env:"DATA_DIR"`
-	ScratchDir  string `env:"SCRATCH_DIR,default=/tmp"`
+	ScratchDir  string `env:"SCRATCH_DIR"`
 	SiteDomain  string `env:"SITE_DOMAIN"`
 	LoginLocked bool   `env:"LOGIN_LOCKED,default=false"`
 	Env         string `env:"ENV,default=prod"`

handlers/middleware/errlog.go

@@ -1,17 +0,0 @@
-package middleware
-
-import (
-	"log"
-
-	"github.com/gofiber/fiber/v3"
-)
-
-func LogErrors() func(c fiber.Ctx) error {
-	return func(c fiber.Ctx) error {
-		if err := c.Next(); err != nil {
-			log.Printf("error: %v\n", err)
-			return err
-		}
-		return nil
-	}
-}

providers/sitebuilder/builder.go

@@ -148,7 +148,7 @@ func (b *Builder) renderFeeds(ctx buildContext, postIter iter.Seq[models.Maybe[*
 			Content: string(renderedPost.HTML),
 			// TO FIX: Created should be first published
 			Created: post.PublishedAt,
-			Updated: post.UpdatedAt,
+			Updated: post.PublishedAt,
 		})
 
 		items++

providers/uploadfiles/provider.go

@@ -26,44 +26,10 @@ func (p *Provider) AdoptFile(site models.Site, up models.Upload, filename string
 		return err
 	}
 
-	if err := os.Rename(filename, fullPath); err == nil {
+	if err := os.Rename(filename, fullPath); err != nil {
+		return err
+	}
 	return nil
-	}
-
-	// Can't rename, possibly because of a cross-link device issue. So copy instead
-	if err := moveFile(filename, fullPath); err != nil {
-		return err
-	}
-
-	return nil
-}
-
-func moveFile(src, dst string) error {
-	if err := copyFile(src, dst); err != nil {
-		_ = os.Remove(dst)
-		return err
-	}
-	_ = os.Remove(src)
-	return nil
-}
-
-func copyFile(src, dst string) error {
-	in, err := os.Open(src)
-	if err != nil {
-		return err
-	}
-	defer in.Close()
-
-	out, err := os.Create(dst)
-	if err != nil {
-		return err
-	}
-	defer out.Close()
-
-	if _, err = io.Copy(out, in); err != nil {
-		return err
-	}
-	return err
 }
 
 func (p *Provider) OpenUpload(site models.Site, up models.Upload) (io.ReadCloser, error) {

services/posts/create.go

@@ -17,8 +17,6 @@ type CreatePostParams struct {
 }
 
 func (s *Service) UpdatePost(ctx context.Context, params CreatePostParams) (*models.Post, error) {
-	now := time.Now()
-
 	site, ok := models.GetSite(ctx)
 	if !ok {
 		return nil, models.SiteRequiredError
@@ -31,14 +29,14 @@ func (s *Service) UpdatePost(ctx context.Context, params CreatePostParams) (*mod
 
 	post.Title = params.Title
 	post.Body = params.Body
-	post.UpdatedAt = now
+	post.UpdatedAt = time.Now()
 	post.Slug = post.BestSlug()
 	oldState := post.State
 
 	switch strings.ToLower(params.Action) {
 	case "publish":
 		post.State = models.StatePublished
-		post.PublishedAt = now
+		post.PublishedAt = time.Now()
 	case "save draft":
 		post.State = models.StateDraft
 		post.PublishedAt = time.Time{}