I ran into an issue where someone on my team was trying to create a subspace using a blueprint whose "space to clone" was the parent space, i.e. trying to clone the space into itself. Unsurprisingly, this didn't go well. At best, it led to a 502 error, at worst it created a nested string of spaces 100+ deep. (How the latter happened, I have no idea, I couldn't replicate that).

Obviously, people shouldn't really be doing this anyway, but better for thing not to break if they do. The patch below adds a check to avoid recursion in this scenario.

CommentFileSizeAuthor
#2 oa_clone-infinite-loop-2718857-2.patch750 bytesJeffM2001
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JeffM2001 created an issue. See original summary.

JeffM2001’s picture

Status: Active » Needs review
FileSize
750 bytes
mpotter’s picture

Status: Needs review » Fixed

Committed to f6e1afc in oa_clone

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.