Fixed access management issue with URLComponents.

[awelch6]

Instead of overwriting the existing urlComponents.queryItems with itself and then adding the new queryItems variable on top of it. I appended the contents of the queryItems variable to urlComponents.queryItems. This fixes the access management issues, and results in slightly cleaner code.

[max-vyshnevskyi]

@awelch6, your changes will break SDK.
urlComponents?.queryItems?.append(contentsOf: queryItems) is not the same as urlComponents?.queryItems = (urlComponents?.queryItems ?? []) + queryItems.
If urlComponents.queryItems will be nil (spoiler: they will), then after appending it still be nil, but after assigning it will have a value.
Unfornaturely, you have to create a local variable.