Infinite loop when creating email_tfa_status field resulting in nginx 502

Created on 26 October 2022, over 1 year ago
Updated 25 January 2023, over 1 year ago

Problem/Motivation

Combining Email TFA module(release 1.0.5) with Field Group module (release 3.2.0) results in a 502 nginx error when accessing /admin/people/create or /user/XX/edit?destination=/admin/people due to an infinite loop in field_group_field_info_max_weight function provided by Field Group module.

In email_tfa_entity_base_field_info function, provided by Email TFA module, weight option is not set when field email_tfa_status is created, resulting in the recursive loop previosuly described in Field Group module.

In my opinion, this issue should be solved in Field Group module and not in Email TFA module, but we needed a fast solution for this problem

Steps to reproduce

drupal core version: 9.4.8
PHP version: 7.4.24
Email TFA module: 1.0.5
Field Group module: release 3.2.0

When accessing /admin/people/create or /user/XX/edit?destination=/admin/people, 502 nginx error

Proposed resolution

Temporal solution: creating a patch that sets a weight option in email_tfa_status newly created field in Email TFA module. As i said before, I think a deeper solution should be implemented directly in the root of the problem: Field Group module -or maybe core, I am not sure-.

Remaining tasks

Trying to offer a deeper solution directly in Field Group module or core

πŸ› Bug report
Status

Fixed

Version

1.0

Component

Code

Created by

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.69.0 2024