mParticle / mparticle-dotnet-sdk

2 stars 3 forks source link

Handle 429 status code rate limiting for the logEvent/UploadQueue code path #10

Closed einsteinx2 closed 2 years ago

einsteinx2 commented 2 years ago

Summary

Handle 429 status code rate limiting for the logEvent/UploadQueue code path. Note this is NOT the s2s batch upload code path (as seen in our other s2s SDKs).

This is basically the same logic as the s2s 429 handling in that it calculates a timestamp, but since we handle the uploading internally and don't return a response object to the user, it's simpler.

This PR also updates the Newtonsoft.Json library since it had a recent security issue, as well as updates the .NET SDKs to versions that are still supported, which allowed for some cleanup in the CI actions (no longer need to install .NET Core 2.1).

NOTE: THIS IS MEANT TO BE REBASE MERGED NOT SQUASHED AS EACH COMMIT SHOULD STAND ON IT'S OWN IN THE GIT HISTORY

Testing Plan

Unit tests were added and local testing was performed.

Master Issue

Closes https://go.mparticle.com/work/SQDSDKS-4229