Skip to content

Bugfix: avoid 500 when resending confirmation

Jay requested to merge jswain_almost_there_bug into master

What does this MR do?

The resource argument in the after_resending_confirmation_instructions_path_for method is actuall a symbol, not a user object. Avoiding calling methods on this argument when redirecting to the almost_there path

note: the almost_there page renders gracefully without an email param.

part of: #333693 (closed) (comment 602683385)

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team
Edited by Mayra Cabrera

Merge request reports

Loading