yesodweb/yesod

Breadcrumbs: Warn against infinite loop

NorfairKing opened this issue · 2 comments

Hi! Every once in a while I accidentally make an infinite loop in breadcrumbs, which then locks up my computer entirely.
Is there any chance we could give this code a recursion limit or a loop detector?

go back Nothing = return back
go back (Just this) = do
(title, next) <- breadcrumb this
go ((this, title) : back) next

Seems OK, PR welcome.

jezen commented

I believe this is fixed in #1728 so I'll close this issue.