fix: fix replication of multiple projects with numeric names

[cgirard-mir]

Thank you for contributing to Harbor!

Comprehensive Summary of your change

The original fix to #19027 contained in #19090 worked well for replicating a single project whose name was numeric, but did not resolve the issue when multiple numeric project names were provided.

The issue was in the construction of the API call. The query parameter was previously being constructed as name={'123 456 789'}. With the addition of some debug logging (also included in this PR), I noticed that the server was parsing this as '123, 456, 789'. Note that the single quotes were being associated with the first and last values of the list, so these values were forced into parsing as string values. However, the middle value contains only numbers and so was parsed as such. This doesn't cause a problem when only 1 or 2 values are being passed, but can show itself once there are 3 or more values.

The solution is to surround each project name in quotes, name={'123' '456' '789'}, so they are all handled as strings by the server.

Issue being fixed

Further fixes #19027.

Please indicate you've done the following:

• Well Written Title and Summary of the PR
• Label the PR as needed. "release-note/ignore-for-release, release-note/new-feature, release-note/update, release-note/enhancement, release-note/community, release-note/breaking-change, release-note/docs, release-note/infra, release-note/deprecation"
• Accepted the DCO. Commits without the DCO will delay acceptance.
• Made sure tests are passing and test coverage is added if needed.
• Considered the docs impact and opened a new docs issue or PR with docs changes if needed in website repository.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 46.17%. Comparing base (c8c11b4) to head (77da6b0).
Report is 382 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #21474      +/-   ##
==========================================
+ Coverage   45.36%   46.17%   +0.81%     
==========================================
  Files         244      247       +3     
  Lines       13333    13883     +550     
  Branches     2719     2875     +156     
==========================================
+ Hits         6049     6411     +362     
- Misses       6983     7134     +151     
- Partials      301      338      +37     

Flag	Coverage Δ
unittests	46.17% <ø> (+0.81%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 491 files with indirect coverage changes

[Vad1mo]

nice catch!

[cgirard-mir]

FYI, all of the tests were previously passing on this, but after I updated the branch for "out-of-date with base", the test statuses have been reset. Since I don't have permission to run those tests, I'm not planning to do another "Update branch" until they've been re-run and the PR has 2 approvals from maintainers.

[chlins]

lgtm