Skip to content

Commit

Permalink
fixup! provisioning/LinkNew: use structured logging
Browse files Browse the repository at this point in the history
Signed-off-by: Sumner Evans <[email protected]>
  • Loading branch information
sumnerevans committed Dec 30, 2023
1 parent df8684f commit a353ef0
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions provisioning.go
Original file line number Diff line number Diff line change
Expand Up @@ -352,13 +352,15 @@ func (prov *ProvisioningAPI) clearSession(user *User) {

// ** Provisioning API Helpers ** //

func (prov *ProvisioningAPI) loginOrSendError(w http.ResponseWriter, user *User) (*provisioningHandle, error) {
func (prov *ProvisioningAPI) loginOrSendError(ctx context.Context, w http.ResponseWriter, user *User) (*provisioningHandle, error) {
newSessionLoggedIn, handle, err := prov.newOrExistingSession(user)
if err != nil {
return nil, err
}
if !newSessionLoggedIn {
prov.log.Debug().Msgf("LinkNew from %v, user already has a pending provisioning request (%d), cancelling", user.MXID, handle.id)
zerolog.Ctx(ctx).Debug().
Int("existing_provisioning_handle", handle.id).
Msg("user already has pending provisioning request, cancelling")
prov.clearSession(user)
newSessionLoggedIn, handle, err = prov.newOrExistingSession(user)
if err != nil {
Expand Down Expand Up @@ -400,9 +402,10 @@ func (prov *ProvisioningAPI) LinkNew(w http.ResponseWriter, r *http.Request) {
Str("action", "link_new").
Str("user_id", user.MXID.String()).
Logger()
ctx := log.WithContext(r.Context())
log.Debug().Msg("starting login")

handle, err := prov.loginOrSendError(w, user)
handle, err := prov.loginOrSendError(ctx, w, user)
if err != nil {
jsonResponse(w, http.StatusInternalServerError, Error{
Success: false,
Expand Down

0 comments on commit a353ef0

Please sign in to comment.