Room remains open after destroy

[ilDuna]

Description:

We have a custom prosody module which triggers a room destroy under certain conditions.
With latest Prosody versions, when the room is re-opened after a destroy, it remains open also if every participant leaves.

Steps to reproduce:

1. User A opens a room room-name
2. User B joins the room room-name
3. Using the prosody console on the server (prosodyctl shell) we trigger a room destroy using the command

muc:room("<room-name>@conference.<jitsi-domain>"):destroy()

4. User A and User B join again the same room
5. Room is closed normally (hit the red button or close the tab)
6. User A joins again the same room

Expected behavior:

1. The room timer starts from 00:00

Actual behavior:

The room timer starts from the last value. If we close again the room, the room timer does not reset.
Furthermore, if we enable the muc_census plugin on Prosody, we can see that when every participants quit, the room is still there!

# curl -H "Host: <jitsi-domain>" http://localhost:5280/room-census && echo ""
{
  "room_census":[
    {
      "created_time":1650548064000,
      "participants":0,
      "room_name":"<room-name>@conference.<jitsi-domain>"
    }
  ]
}

Server information:

• Jitsi Meet version:

ii  jicofo                           1.0-874-1            
ii  jitsi-meet                       2.0.7210-1           
ii  jitsi-meet-prosody               1.0.6091-1           
ii  jitsi-meet-turnserver            1.0.6091-1           
ii  jitsi-meet-web                   1.0.6091-1           
ii  jitsi-meet-web-config            1.0.6091-1           
ii  jitsi-videobridge2               2.1-665-g3a90ccdc-1  
ii  prosody                          0.12.0-1~focal1      

• Operating System: Ubuntu Server 20.04.4 LTS

Client information:

• Browser / app version: Chrome
• Operating System: Ubuntu Desktop 20.04 LTS

Additional information:

Same issue with stable/jitsi-meet_7001 and latest prosody 0.11

[damencho]

I'm seeing jicofo staying in the room. room_census filter jicofo.

[damencho]

The destroy should be like this for the moment:

muc:room("<room-name>@conference.<jitsi-domain>"):destroy(nil, "We are destroying this room!")

Till we make the reason optional for jicofo. But still there is a the problem you describe.

[damencho]

I'm looking into this.

[ilDuna]

The destroy should be like this for the moment:

muc:room("<room-name>@conference.<jitsi-domain>"):destroy(nil, "We are destroying this room!")

Till we make the reason optional for jicofo. But still there is a the problem you describe.

Yep, I'm already using also the reason argument inside the prosody plugin