Force POST for bulk requests, fix alwaysUseSendBeacon not respected for bulk requests #15784
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fix #15740
fix #14512
Noticed bulk requests weren't working when they had less than 2000 characters.
While fixing the issue also noticed that the
alwaysUseSendBeacon
flag was not respected for bulk requests so added them as well. Now we're making sure that any bulk request will be POSTed and not used as a URL parameter.As a result, bulk requests were considered page views with an empty page title instead of the actual requests. Noticed this issue in almost all our websites.