Mailbox/set httpd endless loop

[zvasilev]

If you call Mailbox/set create and set non existent reference "#" for the 'parentId' parameter, httpd enter in endless loop.

[rsto]

Thanks for reporting this! It even looped for all requests where there's only missing creationIds in parentId, so that really needed a fix.

[rsto]

Tested in cyrusimap/cassandane@3190f45

[zvasilev]

Hello Robert,

Thank for fix the issue!

I have a problem with creating subfolders. Seems "Mailbox/set" doesn't honor "parentId" argument.
For example, when I try to create "Inbox" subfolder and pass inbox mailbox id, the folder is created but as a sibling of the inbox.
When I look at the code, I just saw it tries only if starts with '#' char.
Still not sure Am I read RFC correctly or is Cyrus bug.

My JSON payload is
{"methodCalls":[["Mailbox/set",{"create":{"1":{"name":"folder_123","parentId":"c8f73a59-4b7c-4549-8c03-0e267583b4e2"}}},"#13"]],"using":["ietf:jmap","ietf:jmapmail"]}

[rsto]

@zvasilev Your JMAP request looks OK to me.

Could you please create a new issue including the Mailbox/set request and response, and the response of a subsequent Mailbox/get for the two mailboxes?