Skip to content

Draft: Migration: remove unused column verification_attempts from user_phone_number_validations

Hinam Mehra requested to merge 79-anti-abuse-remove-verification-attempts-column into master

What does this MR do and why?

I created a user_phone_number_validations table !97762 (diffs), with a verification_attempts column. This column was initially intended to rate-limit the amount of tries a user gets to verify their phone number. But this isn’t needed anymore since we are going to be throttling with ApplicationRateLimiter !103429 (diffs) and I would like to remove it.

The phone-verification feature is behind a feature flag & hasn’t been rolled out to any users.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to https://gitlab.com/gitlab-org/modelops/anti-abuse/team-tasks/-/issues/79

Edited by Hinam Mehra

Merge request reports