Always comes an empty tracking fields array

[SilverrainSPB]

When calling the ZoomClient method.Meetings.GetAsync always receives an empty TrackingFields array.
The problem clearly lies in the parsing of the incoming response.

[SilverrainSPB]

The problem is that the first StartObject token is always skipped due to calling reader.Read() before the loop and calling it again in the loop condition. Therefore, you can simply remove reader.Read() before the loop and move the reader.Read() call to the beginning of the loop condition.

I can't create a branch and pull request with the necessary fix. But Read method of KeyValuePairConverter should look like this:

		public override KeyValuePair<string, string>[] Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
		{
			if (reader.TokenType == JsonTokenType.StartArray)
			{
				var values = new List<KeyValuePair<string, string>>();

				while (reader.Read() && (reader.TokenType != JsonTokenType.EndArray))
				{
					if (reader.TokenType == JsonTokenType.StartObject)
					{
						var fieldName = string.Empty;
						var fieldValue = string.Empty;

						while ((reader.TokenType != JsonTokenType.EndObject) && reader.Read())
						{
							if (reader.TokenType == JsonTokenType.PropertyName)
							{
								var propertyName = reader.GetString();
								reader.Read();

								if (propertyName == _keyFieldName) fieldName = reader.GetString();
								else if (propertyName == _valueFieldName) fieldValue = reader.GetString();
							}
						}

						values.Add(new KeyValuePair<string, string>(fieldName, fieldValue));
					}
				}

				return values.ToArray();
			}

			throw new Exception("Unable to read Key/Value pair");
		}

Please check it and release the new version of the ZoomNet lib as soon as it possible. I'm realy looking forward to it. :)

[Jericho]

Thanks for reporting this issue. I was able to confirm there's an issue in the custom JSON converter that causes the first item in the array to be skipped. In other words:

• If the JSON returned by the Zoom API contains three tracking fields, the resulting array parsed by our custom converter will contain 2 items
• if the JSON contains two tracking fields, the parsed array will contain 1 item
• if the JSON contains a single tracking field (I suspect this is your case), the parsed array will be empty

So the resulting array is not always empty, but if your JSON only contains a single tracking field I can see how you would reach this conclusion.

Regardless, this bug needs to be fixed.

[SilverrainSPB]

That's right, I tested this method when using a single tracking_field.

Anyway, thank you for the fast answer and commit.

[Jericho]

Package is being built as I write this. Will be on Nuget shortly (probably in an hour or so... depending how long it takes for Nuget to index the package)

[Jericho]

🎉 This issue has been resolved in version 0.63.1 🎉

The release is available on:

• GitHub Release
• NuGet Package

Your GitReleaseManager bot 📦🚀