lfs: do not require auth for public repositories

What does this MR do?

Allow LFS objects on public repos to be downloaded without authentication.

Are there points in the code the reviewer needs to double check?

Maybe?

Why was this MR needed?

LFS downloads on public repositories is obnoxious without this.

Does this MR meet the acceptance criteria?

• Changelog entry added
• Documentation created/updated
• Tests
  • Added for this feature/bug
  • All builds are passing
• Conform by the merge request performance guides
• Conform by the style guides
• Branch has no merge conflicts with master (if it does - rebase it please)
• Squashed related commits together

What are the relevant issue numbers?

#23360 (closed)

added 1 commit

• 21f09442 - lfs: do not require auth for public repositories

Compare with previous version

mentioned in issue #23360 (closed)

changed title from lfs: allow public repos to download data just fine to lfs: do not require auth for public repositories

resolved all discussions

added 1 commit

• 6df7e2e0 - lfs: do not require auth for public repositories

Compare with previous version

added 1 commit

• e943e9b3 - lfs: do not require auth for public repositories

Compare with previous version

marked the task Branch has no merge conflicts with master (if it does - rebase it please) as completed

I don't know how to interpret the failures. Are they my fault?

marked the task Added for this feature/bug as completed

Ping? @stanhu Can you interpret the test results for me?

Don't think so--you might want to retry failed.

marked the task All builds are passing as completed

All tests are passing.

assigned to @rymai

added ~164274 lfs labels

added ~480950 label

@ben.boeckel Thanks, I had one remark, LGTM otherwise!

assigned to @ben.boeckel

added 1 commit

• 5c20e50f - lfs: do not require auth for public repositories

Compare with previous version

resolved all discussions

marked the task All builds are passing as incomplete

marked the task All builds are passing as completed

Tests are passing.

@ben.boeckel Awesome work, thanks a lot!

added ~149423 label

changed milestone to %8.17

assigned to @rymai

merged

mentioned in commit 1a1381a4

removed ~149423 label

Picked into 8-17-stable, will go with 8.17 RC3