diff --git a/main.go b/main.go index 2ff81c1..05f0a7d 100644 --- a/main.go +++ b/main.go @@ -475,6 +475,9 @@ func (u *ui) describeActionError(label string, err error) string { if errors.Is(err, webdav.ErrConflict) || strings.Contains(err.Error(), webdav.ErrConflict.Error()) { return "Save conflict: the remote vault changed. Reopen it and retry the save." } + if label == "open remote vault" { + return fmt.Sprintf("%s failed: %v", label, err) + } return err.Error() } @@ -736,6 +739,8 @@ func (u *ui) header(gtx layout.Context) layout.Dimensions { }), layout.Rigid(layout.Spacer{Height: unit.Dp(8)}.Layout), layout.Rigid(u.lifecycleControls), + layout.Rigid(layout.Spacer{Height: unit.Dp(8)}.Layout), + layout.Rigid(u.feedbackBanner), ) }) } @@ -766,6 +771,8 @@ func (u *ui) header(gtx layout.Context) layout.Dimensions { }), layout.Rigid(layout.Spacer{Height: unit.Dp(10)}.Layout), layout.Rigid(u.lifecycleControls), + layout.Rigid(layout.Spacer{Height: unit.Dp(10)}.Layout), + layout.Rigid(u.feedbackBanner), ) }) } @@ -1303,6 +1310,27 @@ func detailLine(th *material.Theme, label, value string) layout.Widget { } } +func (u *ui) feedbackBanner(gtx layout.Context) layout.Dimensions { + message := u.statusMessage + tone := color.NRGBA{R: 231, G: 239, B: 235, A: 255} + textColor := accentColor + if u.errorMessage != "" { + message = u.errorMessage + tone = color.NRGBA{R: 248, G: 226, B: 223, A: 255} + textColor = color.NRGBA{R: 140, G: 46, B: 34, A: 255} + } + if message == "" { + return layout.Dimensions{} + } + return layout.Background{}.Layout(gtx, fill(tone), func(gtx layout.Context) layout.Dimensions { + return layout.UniformInset(unit.Dp(10)).Layout(gtx, func(gtx layout.Context) layout.Dimensions { + lbl := material.Label(u.theme, unit.Sp(13), message) + lbl.Color = textColor + return lbl.Layout(gtx) + }) + }) +} + func (u *ui) passwordLine(label, value string) layout.Widget { return func(gtx layout.Context) layout.Dimensions { icon := u.eyeIcon diff --git a/main_test.go b/main_test.go index e1cba08..bc43936 100644 --- a/main_test.go +++ b/main_test.go @@ -560,6 +560,99 @@ func TestUIOpenRemoteAndSaveThroughConfiguredWebDAVTarget(t *testing.T) { } } +func TestUIOpenRemoteReportsTransportFailure(t *testing.T) { + t.Parallel() + + server := httptest.NewServer(http.HandlerFunc(func(http.ResponseWriter, *http.Request) {})) + url := server.URL + server.Close() + + u := newUIWithSession("desktop", &session.Manager{}) + u.masterPassword.SetText("correct horse battery staple") + u.remoteBaseURL.SetText(url) + u.remotePath.SetText("vaults/main.kdbx") + + u.runAction("open remote vault", u.openRemoteAction) + + if got := u.errorMessage; !strings.Contains(got, "open remote vault failed:") { + t.Fatalf("errorMessage = %q, want open remote vault failure", got) + } + if got := u.statusMessage; got != "" { + t.Fatalf("statusMessage = %q, want empty on remote open failure", got) + } +} + +func TestUIRemoteSaveConflictShowsVisibleErrorAndKeepsDirtyState(t *testing.T) { + t.Parallel() + + key := vault.MasterKey{Password: "correct horse battery staple"} + model := vault.Model{ + Entries: []vault.Entry{ + { + ID: "vault-console", + Title: "Vault Console", + Username: "dannyocean", + Password: "token-1", + URL: "https://vault.crew.example.invalid", + Path: []string{"Root", "Internet"}, + }, + }, + } + + var putCount int + server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + switch r.Method { + case http.MethodGet: + var encoded bytes.Buffer + if err := vault.SaveKDBXWithKey(&encoded, model, key); err != nil { + t.Fatalf("SaveKDBXWithKey() error = %v", err) + } + w.Header().Set("ETag", "\"v1\"") + _, _ = w.Write(encoded.Bytes()) + case http.MethodPut: + putCount++ + w.WriteHeader(http.StatusPreconditionFailed) + default: + t.Fatalf("unexpected method %s", r.Method) + } + })) + defer server.Close() + + u := newUIWithSession("desktop", &session.Manager{}) + u.masterPassword.SetText("correct horse battery staple") + u.remoteBaseURL.SetText(server.URL) + u.remotePath.SetText("vaults/main.kdbx") + + if err := u.openRemoteAction(); err != nil { + t.Fatalf("openRemoteAction() error = %v", err) + } + if err := u.state.UpsertEntry(vault.Entry{ + ID: "vault-console", + Title: "Vault Console", + Username: "dannyocean", + Password: "token-2", + URL: "https://vault.crew.example.invalid", + Path: []string{"Root", "Internet"}, + }); err != nil { + t.Fatalf("UpsertEntry() error = %v", err) + } + + u.runAction("save vault", u.saveAction) + + if got := u.errorMessage; got != "Save conflict: the remote vault changed. Reopen it and retry the save." { + t.Fatalf("errorMessage = %q, want normalized save conflict guidance", got) + } + if got := u.statusMessage; got != "" { + t.Fatalf("statusMessage = %q, want empty after remote save conflict", got) + } + if !u.state.Dirty { + t.Fatal("Dirty = false, want true after remote save conflict") + } + if putCount != 1 { + t.Fatalf("remote PUT count = %d, want 1", putCount) + } +} + func TestUIMasterKeyInputSupportsKeyFileAndCompositeKeys(t *testing.T) { t.Parallel()