WIP: Allow setting autosync.interval in minutes with "m" suffix

Solves #2730

Author: quite

docs/config.md

@@ -84,7 +84,7 @@ This is a list of available options:
 | `audit.concurrency`             | `int`    | Number of concurrent audit workers.                                                                                                                                                                                                | ``                                  |
 | `audit.hibp-dump-file`          | `string` | Specify to a HIBPv2 Dump file (sorted) if you want `audit` to check password hashes against this file.                                                                                                                             | `None`                              |
 | `audit.hibp-use-api`            | `bool`   | Set to true if you want `gopass audit` to check your secrets against the public HIBPv2 API. Use with caution. This will leak a few bit of entropy.                                                                                 | `false`                             |
-| `autosync.interval`             | `int`    | AutoSync interval in days.                                                                                                                                                                                                         | `3`                                 |
+| `autosync.interval`             | `string` | AutoSync interval in days, or minutes like "3m".                                                                                                                                                                                   | `3`                                 |
 | `core.autoimport`               | `bool`   | Import missing keys stored in the pass repository without asking.                                                                                                                                                                  | `false`                             |
 | `core.autopush`                 | `bool`   | Always do a `git push` after a commit to the store. Makes sure your local changes are always available on your git remote.                                                                                                         | `true`                              |
 | `core.autosync`                 | `bool`   | Automatically sync (fetch & push) the git remote on an interval.                                                                                                                                                                   | `true`                              |

internal/action/sync.go

@@ -6,9 +6,11 @@ import (
 	"fmt"
 	"os"
 	"strconv"
+	"strings"
 	"time"
 
 	"github.com/fatih/color"
+	"github.com/gopasspw/gopass/internal/action/exit"
 	"github.com/gopasspw/gopass/internal/backend"
 	"github.com/gopasspw/gopass/internal/config"
 	"github.com/gopasspw/gopass/internal/diff"
@@ -22,9 +24,11 @@ import (
 	"github.com/urfave/cli/v2"
 )
 
+const dayInMinutes = 24 * 60
+
 var (
-	autosyncIntervalDays = 3
-	autosyncLastRun      time.Time
+	autosyncIntervalMinutes = 3 * dayInMinutes
+	autosyncLastRun         time.Time
 )
 
 func init() {
@@ -40,7 +44,7 @@ func init() {
 		return
 	}
 
-	autosyncIntervalDays = iv
+	autosyncIntervalMinutes = iv * dayInMinutes
 }
 
 // Sync all stores with their remotes.
@@ -69,13 +73,26 @@ func (s *Action) autoSync(ctx context.Context) error {
 
 	ls := s.rem.LastSeen("autosync")
 	debug.Log("autosync - last seen: %s", ls)
-	syncInterval := autosyncIntervalDays
+	syncInterval := autosyncIntervalMinutes
 
 	if s.cfg.IsSet("autosync.interval") {
-		syncInterval = s.cfg.GetInt("autosync.interval")
+		str := s.cfg.Get("autosync.interval")
+		if strings.HasSuffix(str, "m") {
+			minutes, err := strconv.Atoi(strings.TrimSuffix(str, "m"))
+			if err != nil {
+				return exit.Error(exit.Usage, err, "failed to parse autosync.interval %q: %q", str, err)
+			}
+			syncInterval = minutes
+		} else {
+			days, err := strconv.Atoi(str)
+			if err != nil {
+				return exit.Error(exit.Usage, err, "failed to parse autosync.interval %q: %q", str, err)
+			}
+			syncInterval = days * dayInMinutes
+		}
 	}
 
-	if time.Since(ls) > time.Duration(syncInterval)*24*time.Hour {
+	if time.Since(ls) > time.Duration(syncInterval)*time.Minute {
 		err := s.sync(ctx, "")
 		if err != nil {
 			autosyncLastRun = time.Now()