Draft: Geo - Fix syncing remote stored Blobs with filenames with plus sign

Douglas Barbosa Alexandrerequested to merge
456901-geo-filenames-with-plus-sign-fail-to-sync-with-non-success-http-response-status-code-403 into master

What does this MR do and why?

The http gem can re-encode or normalize URLs in ways that break AWS S3 signatures. When it sees a + or %2B in the pre-signed URL, it might:

  • Convert + to %20 (space)
  • Re-encode already encoded characters
  • Normalize the URL in ways that change the signature

Since there is no easy way to disable the URL normalization in the http gem to fix the underlying issue without introducing regressions like !55775 (merged), this MR replaces it with the httparty gem. The HTTParty can re-encode or normalize URLs in the same way, which breaks the AWS S3 signatures, but it has a simple way to disable the URL normalization.

How to set up and validate locally

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Related issues

Related to #456901 (closed)

Edited by Douglas Barbosa Alexandre

Merge request reports