aboutsummaryrefslogtreecommitdiff
path: root/internal/reader/opml/handler.go
diff options
context:
space:
mode:
authorGravatar Frédéric Guillot <f@miniflux.net> 2023-09-24 16:32:09 -0700
committerGravatar Frédéric Guillot <f@miniflux.net> 2023-09-24 22:37:33 -0700
commitc0e954f19d707fef8ef8271636ec661634a4c4c7 (patch)
tree5aa052a048f470e233a454e5ad9071eed1fa37c0 /internal/reader/opml/handler.go
parent54cb8fa0286e4a2f1a81c32b5a89722d93b30bf7 (diff)
downloadv2-c0e954f19d707fef8ef8271636ec661634a4c4c7.tar.gz
v2-c0e954f19d707fef8ef8271636ec661634a4c4c7.tar.zst
v2-c0e954f19d707fef8ef8271636ec661634a4c4c7.zip
Implement structured logging using log/slog package
Diffstat (limited to 'internal/reader/opml/handler.go')
-rw-r--r--internal/reader/opml/handler.go11
1 files changed, 3 insertions, 8 deletions
diff --git a/internal/reader/opml/handler.go b/internal/reader/opml/handler.go
index 1d874a87..c3e972b9 100644
--- a/internal/reader/opml/handler.go
+++ b/internal/reader/opml/handler.go
@@ -4,11 +4,9 @@
package opml // import "miniflux.app/v2/internal/reader/opml"
import (
- "errors"
"fmt"
"io"
- "miniflux.app/v2/internal/logger"
"miniflux.app/v2/internal/model"
"miniflux.app/v2/internal/storage"
)
@@ -53,21 +51,18 @@ func (h *Handler) Import(userID int64, data io.Reader) error {
if subscription.CategoryName == "" {
category, err = h.store.FirstCategory(userID)
if err != nil {
- logger.Error("[OPML:Import] %v", err)
- return errors.New("unable to find first category")
+ return fmt.Errorf("opml: unable to find first category: %w", err)
}
} else {
category, err = h.store.CategoryByTitle(userID, subscription.CategoryName)
if err != nil {
- logger.Error("[OPML:Import] %v", err)
- return errors.New("unable to search category by title")
+ return fmt.Errorf("opml: unable to search category by title: %w", err)
}
if category == nil {
category, err = h.store.CreateCategory(userID, &model.CategoryRequest{Title: subscription.CategoryName})
if err != nil {
- logger.Error("[OPML:Import] %v", err)
- return fmt.Errorf(`unable to create this category: %q`, subscription.CategoryName)
+ return fmt.Errorf(`opml: unable to create this category: %q`, subscription.CategoryName)
}
}
}