heimoshuiyu/postmoogle
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Suppress M_NOT_FOUND error when fetching settings

Browse Source 
.. and ensure that getSettings() always returns either a default empty
settings object or an error.
This commit is contained in:
Slavi Pantaleev
2022-08-23 16:30:19 +03:00
parent b24bd498a0
commit 726bc95c26
3 changed files with 18 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
bot/owner.go
View File

@@ -23,7 +23,8 @@ func (b *Bot) getOwner(ctx context.Context, evt *event.Event) {
cfg, err := b.getSettings(span.Context(), evt.RoomID)
if err != nil {
b.Error(span.Context(), evt.RoomID, "owner is not set yet")
b.log.Warn("cannot get settings: %v", err)
b.Error(span.Context(), evt.RoomID, "failed to retrieve settings")
return
}
@@ -48,6 +49,8 @@ func (b *Bot) setOwner(ctx context.Context, evt *event.Event, owner string) {
cfg, err := b.getSettings(span.Context(), evt.RoomID)
if err != nil {
b.log.Warn("cannot get settings: %v", err)
b.Error(span.Context(), evt.RoomID, "failed to retrieve settings")
return
}
if !cfg.Allowed(b.noowner, evt.Sender) {