Skip to content

LG-8763 Log that a profile is pending on password reset#8872

Merged
jmhooper merged 2 commits intomainfrom
jmhooper-log-that-profile-is-pending-on-password-reset
Jul 26, 2023
Merged

LG-8763 Log that a profile is pending on password reset#8872
jmhooper merged 2 commits intomainfrom
jmhooper-log-that-profile-is-pending-on-password-reset

Conversation

@jmhooper
Copy link
Contributor

When a user resets their password it deactivates their active profile. We log that we do that.

Additionally, we invalidate any pending profiles which will encounter a encryption error when the user logs in. This commit adds logging to log that a pending profile is present and the reasons that it is pending so we can measure the impact of this behavior at password reset

jmhooper added 2 commits July 26, 2023 13:59
When a user resets their password it deactivates their active profile. We log that we do that.

Additionally, we invalidate any pending profiles which will encounter a encryption error when the user logs in. This commit adds logging to log that a pending profile is present and the reasons that it is pending so we can measure the impact of this behavior at password reset

[skip changelog]
@jmhooper jmhooper requested a review from a team July 26, 2023 19:10
Copy link
Contributor

@soniaconnolly soniaconnolly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jmhooper jmhooper merged commit e89448f into main Jul 26, 2023
@jmhooper jmhooper deleted the jmhooper-log-that-profile-is-pending-on-password-reset branch July 26, 2023 19:29
mitchellhenke pushed a commit that referenced this pull request Jul 26, 2023
* LG-8763 Log that a profile is pending on password reset

When a user resets their password it deactivates their active profile. We log that we do that.

Additionally, we invalidate any pending profiles which will encounter a encryption error when the user logs in. This commit adds logging to log that a pending profile is present and the reasons that it is pending so we can measure the impact of this behavior at password reset

[skip changelog]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants