For actions like join/leave/refresh, the loop server logging correctly includes a "roomConnectionId" field. However, for the "status" action, this field isn't currently appearing in the log output. Please update the logging code to include this field for all actions, and "status" in particular. I recognize that this operation will likely require retrieving the room participant information from the data store; however, without this correlation, it is difficult to make use of the status information. Once we implement the link clicker parity feature, it will become impossible to distinguish information from different clients, making the "status" data effectively worthless unless we also have roomConnectionId. In other words, I believe that the value of this data is well worth the cost of another database hit.
Created attachment 8633989 [details] [review] Link to Github PR — #339.
Assignee: nobody → rhubscher
Status: NEW → ASSIGNED
Comment on attachment 8633989 [details] [review] Link to Github PR — #339. This looks pretty much like what I expected. I have a couple of nits that I've pointed out on github.
Attachment #8633989 - Flags: feedback?(adam) → feedback+
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Attachment #8633989 - Flags: review?(mathieu) → review+
You need to log in before you can comment on or make changes to this bug.