chore: replace log/slog with standard logger

.github/workflows/build.yml

@@ -43,7 +43,9 @@ jobs:
           diff -u <(echo -n) <(gofmt -d .)
 
       - name: Get dependencies
-        run: go mod download
+        run: |
+          go mod tidy
+          go mod download
 
       - name: Build
         run: go build -v ./...

.github/workflows/lint.yml

@@ -24,6 +24,11 @@ jobs:
         with:
           config: ./.typos.toml
 
+      - name: Get dependencies
+        run: |
+          go mod tidy
+          go mod download
+
       - name: Lint
         uses: golangci/golangci-lint-action@v6
         with:

.gitignore

@@ -205,4 +205,6 @@ hack-browser-data
 
 # CICD Config
 !.typos.toml
-!.github/*.yml
+!.github/*.yml
+!log/
+examples/*.go

.golangci.yml

@@ -1,7 +1,7 @@
 run:
   timeout: '5m'
   allow-parallel-runners: true
-  modules-download-mode: 'readonly'
+  modules-download-mode: 'mod'
 
 linters:
   enable:
@@ -43,11 +43,8 @@ linters:
     - 'noctx'
     - 'sqlclosecheck'
     - 'staticcheck'
-    - 'structcheck'
     - 'stylecheck'
     - 'unused'
-    - 'deadcode'
-    - 'varcheck'
     - 'paralleltest'
 
 issues:

browser/browser.go

@@ -1,14 +1,14 @@
 package browser
 
 import (
-	"log/slog"
 	"path/filepath"
 	"sort"
 	"strings"
 
 	"github.com/moond4rk/hackbrowserdata/browser/chromium"
 	"github.com/moond4rk/hackbrowserdata/browser/firefox"
 	"github.com/moond4rk/hackbrowserdata/browserdata"
+	"github.com/moond4rk/hackbrowserdata/log"
 	"github.com/moond4rk/hackbrowserdata/utils/fileutil"
 	"github.com/moond4rk/hackbrowserdata/utils/typeutil"
 )
@@ -44,16 +44,16 @@ func pickChromium(name, profile string) []Browser {
 	if name == "all" {
 		for _, v := range chromiumList {
 			if !fileutil.IsDirExists(filepath.Clean(v.profilePath)) {
-				slog.Warn("find browser failed, profile folder does not exist", "browser", v.name)
+				log.Warnf("find browser failed, profile folder does not exist, browser %s", v.name)
 				continue
 			}
 			multiChromium, err := chromium.New(v.name, v.storage, v.profilePath, v.dataTypes)
 			if err != nil {
-				slog.Error("new chromium error", "err", err)
+				log.Errorf("new chromium error %v", err)
 				continue
 			}
 			for _, b := range multiChromium {
-				slog.Warn("find browser success", "browser", b.Name())
+				log.Warnf("find browser success, browser %s", b.Name())
 				browsers = append(browsers, b)
 			}
 		}
@@ -63,14 +63,14 @@ func pickChromium(name, profile string) []Browser {
 			profile = c.profilePath
 		}
 		if !fileutil.IsDirExists(filepath.Clean(profile)) {
-			slog.Error("find browser failed, profile folder does not exist", "browser", c.name)
+			log.Errorf("find browser failed, profile folder does not exist, browser %s", c.name)
 		}
 		chromes, err := chromium.New(c.name, c.storage, profile, c.dataTypes)
 		if err != nil {
-			slog.Error("new chromium error", "err", err)
+			log.Errorf("new chromium error %v", err)
 		}
 		for _, chrome := range chromes {
-			slog.Warn("find browser success", "browser", chrome.Name())
+			log.Warnf("find browser success, browser %s", chrome.Name())
 			browsers = append(browsers, chrome)
 		}
 	}
@@ -89,17 +89,17 @@ func pickFirefox(name, profile string) []Browser {
 			}
 
 			if !fileutil.IsDirExists(filepath.Clean(profile)) {
-				slog.Warn("find browser failed, profile folder does not exist", "browser", v.name)
+				log.Warnf("find browser failed, profile folder does not exist, browser %s", v.name)
 				continue
 			}
 
 			if multiFirefox, err := firefox.New(profile, v.dataTypes); err == nil {
 				for _, b := range multiFirefox {
-					slog.Warn("find browser success", "browser", b.Name())
+					log.Warnf("find browser success, browser %s", b.Name())
 					browsers = append(browsers, b)
 				}
 			} else {
-				slog.Error("new firefox error", "err", err)
+				log.Errorf("new firefox error %v", err)
 			}
 		}
 

browser/chromium/chromium.go

@@ -2,13 +2,12 @@ package chromium
 
 import (
 	"io/fs"
-	"log/slog"
 	"os"
 	"path/filepath"
-	"slices"
 	"strings"
 
 	"github.com/moond4rk/hackbrowserdata/browserdata"
+	"github.com/moond4rk/hackbrowserdata/log"
 	"github.com/moond4rk/hackbrowserdata/types"
 	"github.com/moond4rk/hackbrowserdata/utils/fileutil"
 	"github.com/moond4rk/hackbrowserdata/utils/typeutil"
@@ -53,9 +52,12 @@ func (c *Chromium) Name() string {
 
 func (c *Chromium) BrowsingData(isFullExport bool) (*browserdata.BrowserData, error) {
 	// delete chromiumKey from dataTypes, doesn't need to export key
-	dataTypes := slices.DeleteFunc(c.dataTypes, func(i types.DataType) bool {
-		return i == types.ChromiumKey
-	})
+	var dataTypes []types.DataType
+	for _, dt := range c.dataTypes {
+		if dt != types.ChromiumKey {
+			dataTypes = append(dataTypes, dt)
+		}
+	}
 
 	if !isFullExport {
 		dataTypes = types.FilterSensitiveItems(c.dataTypes)
@@ -96,7 +98,7 @@ func (c *Chromium) copyItemToLocal() error {
 			err = fileutil.CopyFile(path, filename)
 		}
 		if err != nil {
-			slog.Error("copy item to local error", "path", path, "filename", filename, "err", err)
+			log.Errorf("copy item to local, path %s, filename %s err %v", path, filename, err)
 			continue
 		}
 	}
@@ -139,7 +141,7 @@ func chromiumWalkFunc(items []types.DataType, multiItemPaths map[string]map[type
 	return func(path string, info fs.FileInfo, err error) error {
 		if err != nil {
 			if os.IsPermission(err) {
-				slog.Warn("skipping walk chromium path permission error", "path", path, "err", err)
+				log.Warnf("skipping walk chromium path permission error, path %s, err %v", path, err)
 				return nil
 			}
 			return err

browser/chromium/chromium_darwin.go

@@ -7,12 +7,12 @@ import (
 	"crypto/sha1"
 	"errors"
 	"fmt"
-	"log/slog"
 	"os"
 	"os/exec"
 	"strings"
 
 	"github.com/moond4rk/hackbrowserdata/crypto"
+	"github.com/moond4rk/hackbrowserdata/log"
 	"github.com/moond4rk/hackbrowserdata/types"
 )
 
@@ -54,6 +54,6 @@ func (c *Chromium) GetMasterKey() ([]byte, error) {
 		return nil, errWrongSecurityCommand
 	}
 	c.masterKey = key
-	slog.Info("get master key success", "browser", c.name)
+	log.Debugf("get master key success, browser %s", c.name)
 	return key, nil
 }

browser/chromium/chromium_linux.go

@@ -5,13 +5,13 @@ package chromium
 import (
 	"crypto/sha1"
 	"fmt"
-	"log/slog"
 	"os"
 
 	"github.com/godbus/dbus/v5"
 	keyring "github.com/ppacher/go-dbus-keyring"
 
 	"github.com/moond4rk/hackbrowserdata/crypto"
+	"github.com/moond4rk/hackbrowserdata/log"
 	"github.com/moond4rk/hackbrowserdata/types"
 )
 
@@ -34,7 +34,7 @@ func (c *Chromium) GetMasterKey() ([]byte, error) {
 	}
 	defer func() {
 		if err := session.Close(); err != nil {
-			slog.Error("close dbus session error", "err", err.Error())
+			log.Errorf("close dbus session error: %v", err)
 		}
 	}()
 	collections, err := svc.GetAllCollections()
@@ -50,7 +50,7 @@ func (c *Chromium) GetMasterKey() ([]byte, error) {
 		for _, i := range items {
 			label, err := i.GetLabel()
 			if err != nil {
-				slog.Warn("get label from dbus", "err", err.Error())
+				log.Warnf("get label from dbus: %v", err)
 				continue
 			}
 			if label == c.storage {
@@ -71,6 +71,6 @@ func (c *Chromium) GetMasterKey() ([]byte, error) {
 	// @https://source.chromium.org/chromium/chromium/src/+/master:components/os_crypt/os_crypt_linux.cc
 	key := crypto.PBKDF2Key(secret, salt, 1, 16, sha1.New)
 	c.masterKey = key
-	slog.Info("get master key success", "browser", c.name)
+	log.Debugf("get master key success, browser %s", c.name)
 	return key, nil
 }

browser/chromium/chromium_windows.go

@@ -5,12 +5,12 @@ package chromium
 import (
 	"encoding/base64"
 	"errors"
-	"log/slog"
 	"os"
 
 	"github.com/tidwall/gjson"
 
 	"github.com/moond4rk/hackbrowserdata/crypto"
+	"github.com/moond4rk/hackbrowserdata/log"
 	"github.com/moond4rk/hackbrowserdata/types"
 	"github.com/moond4rk/hackbrowserdata/utils/fileutil"
 )
@@ -35,9 +35,9 @@ func (c *Chromium) GetMasterKey() ([]byte, error) {
 	}
 	c.masterKey, err = crypto.DecryptWithDPAPI(key[5:])
 	if err != nil {
-		slog.Error("decrypt master key failed", "err", err)
+		log.Errorf("decrypt master key failed, err %v", err)
 		return nil, err
 	}
-	slog.Info("get master key success", "browser", c.name)
+	log.Debugf("get master key success, browser %s", c.name)
 	return c.masterKey, nil
 }

browser/firefox/firefox.go

@@ -6,14 +6,14 @@ import (
 	"errors"
 	"fmt"
 	"io/fs"
-	"log/slog"
 	"os"
 	"path/filepath"
 
 	_ "modernc.org/sqlite" // sqlite3 driver TODO: replace with chooseable driver
 
 	"github.com/moond4rk/hackbrowserdata/browserdata"
 	"github.com/moond4rk/hackbrowserdata/crypto"
+	"github.com/moond4rk/hackbrowserdata/log"
 	"github.com/moond4rk/hackbrowserdata/types"
 	"github.com/moond4rk/hackbrowserdata/utils/fileutil"
 	"github.com/moond4rk/hackbrowserdata/utils/typeutil"
@@ -62,7 +62,7 @@ func firefoxWalkFunc(items []types.DataType, multiItemPaths map[string]map[types
 	return func(path string, info fs.DirEntry, err error) error {
 		if err != nil {
 			if os.IsPermission(err) {
-				slog.Warn("skipping walk firefox path permission error", "path", path, "err", err)
+				log.Warnf("skipping walk firefox path %s permission error: %v", path, err)
 				return nil
 			}
 			return err

browserdata/bookmark/bookmark.go

@@ -2,7 +2,6 @@ package bookmark
 
 import (
 	"database/sql"
-	"log/slog"
 	"os"
 	"sort"
 	"time"
@@ -11,6 +10,7 @@ import (
 	_ "modernc.org/sqlite" // import sqlite3 driver
 
 	"github.com/moond4rk/hackbrowserdata/extractor"
+	"github.com/moond4rk/hackbrowserdata/log"
 	"github.com/moond4rk/hackbrowserdata/types"
 	"github.com/moond4rk/hackbrowserdata/utils/fileutil"
 	"github.com/moond4rk/hackbrowserdata/utils/typeutil"
@@ -111,7 +111,7 @@ func (f *FirefoxBookmark) Extract(_ []byte) error {
 	defer db.Close()
 	_, err = db.Exec(closeJournalMode)
 	if err != nil {
-		slog.Error("close journal mode error", "err", err)
+		log.Errorf("close journal mode error: %v", err)
 	}
 	rows, err := db.Query(queryFirefoxBookMark)
 	if err != nil {
@@ -124,7 +124,7 @@ func (f *FirefoxBookmark) Extract(_ []byte) error {
 			title, url        string
 		)
 		if err = rows.Scan(&id, &url, &bt, &dateAdded, &title); err != nil {
-			slog.Error("scan bookmark error", "err", err)
+			log.Errorf("scan bookmark error: %v", err)
 		}
 		*f = append(*f, bookmark{
 			ID:        id,

browserdata/browserdata.go

@@ -1,9 +1,8 @@
 package browserdata
 
 import (
-	"log/slog"
-
 	"github.com/moond4rk/hackbrowserdata/extractor"
+	"github.com/moond4rk/hackbrowserdata/log"
 	"github.com/moond4rk/hackbrowserdata/types"
 	"github.com/moond4rk/hackbrowserdata/utils/fileutil"
 )
@@ -23,7 +22,7 @@ func New(items []types.DataType) *BrowserData {
 func (d *BrowserData) Recovery(masterKey []byte) error {
 	for _, source := range d.extractors {
 		if err := source.Extract(masterKey); err != nil {
-			slog.Error("parse error", "source_data", source.Name(), "err", err.Error())
+			log.Errorf("parse %s error: %v", source.Name(), err)
 			continue
 		}
 	}
@@ -42,18 +41,18 @@ func (d *BrowserData) Output(dir, browserName, flag string) {
 
 		f, err := output.CreateFile(dir, filename)
 		if err != nil {
-			slog.Error("create file error", "filename", filename, "err", err.Error())
+			log.Errorf("create file %s error: %v", filename, err)
 			continue
 		}
 		if err := output.Write(source, f); err != nil {
-			slog.Error("write to file error", "filename", filename, "err", err.Error())
+			log.Errorf("write to file %s error: %v", filename, err)
 			continue
 		}
 		if err := f.Close(); err != nil {
-			slog.Error("close file error", "filename", filename, "err", err.Error())
+			log.Errorf("close file %s error: %v", filename, err)
 			continue
 		}
-		slog.Warn("export success", "filename", filename)
+		log.Warnf("export success: %s", filename)
 	}
 }
 
@@ -62,7 +61,7 @@ func (d *BrowserData) addExtractors(items []types.DataType) {
 		if source := extractor.CreateExtractor(itemType); source != nil {
 			d.extractors[itemType] = source
 		} else {
-			slog.Debug("source not found", "source", itemType)
+			log.Debugf("source not found: %s", itemType)
 		}
 	}
 }