From 574bf1e47f1b611449faa8fa5848ba63dfc7d0e9 Mon Sep 17 00:00:00 2001 From: jloup Date: Mon, 25 Jun 2018 22:56:45 +0200 Subject: [PATCH] Do not return internal error when SMS send fails. Just log it. --- api/password_reset.go | 2 +- api/user.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/api/password_reset.go b/api/password_reset.go index 0c55c02..0e411f9 100644 --- a/api/password_reset.go +++ b/api/password_reset.go @@ -38,7 +38,7 @@ func (q PasswordResetQuery) Run() (interface{}, *Error) { if CONFIG.FreeSMSUser != "" { err := SendSMS(CONFIG.FreeSMSUser, CONFIG.FreeSMSPass, fmt.Sprintf("'%v' request a password reset. Token '/change-password?token=%v'", q.In.Email, token)) if err != nil { - return nil, NewInternalError(err) + log.Error(err) } } diff --git a/api/user.go b/api/user.go index b285a20..9fca34c 100644 --- a/api/user.go +++ b/api/user.go @@ -115,7 +115,7 @@ func (q SignupQuery) Run() (interface{}, *Error) { if CONFIG.FreeSMSUser != "" { err := SendSMS(CONFIG.FreeSMSUser, CONFIG.FreeSMSPass, fmt.Sprintf("New user signup '%v'", q.In.Email)) if err != nil { - return nil, NewInternalError(err) + log.Error(err) } } -- 2.41.0