Jericho / ZoomNet

.NET client library for the Zoom.us REST API v2
MIT License
68 stars 48 forks source link

`MeetingUpdated.ModifiedFields` includes unmodified fields when parsing webhook event #245

Closed Jericho closed 1 year ago

Jericho commented 1 year ago

The ModifiedFields property is supposed to contain the list of fields that where modified (including the "old" value and the "new" value). However, the json payload received from Zoom includes a few fields that were not modified.

For example, the following snippet shows the json I received when I modified a meeting duration:

"object":
{
    "uuid":"qJTtrovBSPO4ls1q0YxDcw\u003d\u003d",
    "id":83738146014,
    "duration":99,
    "occurrences":[{"occurrence_id":"1668211080000","start_time":"2022-11-11T23:58:00Z"}]
},
"old_object":
{
    "uuid":"qJTtrovBSPO4ls1q0YxDcw\u003d\u003d",
    "id":83738146014,
    "duration":30,
    "occurrences":[{"occurrence_id":"1668211080000","start_time":"2022-11-11T23:58:00Z"}]
},

Notice that this json includes not just the duration, but also uuid, id and occurrences. The unmodified fields should be filtered out when parsing the MeetingUpdated webhook event.

Some of these "unmodified" field such as "id" and "uuid" are useful to identify the meeting that was modified but I question why the "occurences" field in included here

I am assuming the same applies to WebinarUpdated wehbook event as well.

Jericho commented 1 year ago

:tada: This issue has been resolved in version 0.52.0 :tada:

The release is available on:

Your GitReleaseManager bot :package::rocket: