http.bzl : all_urls is given the remote_patch URL which is surely a mistake. #22201
Description
Description of the bug:
all_urls includes the remote_patch URL key for http_archive.
This value is passed to downlolad_and_extract which is a bug.
This means that if the original URLs for the http_archive fail, it will treat the remote_patch as the URL for the archive itself.
Please see:
bazel/tools/build_defs/repo/http.bzl
Line 141 in 0e7c0fb
This came up during my writing of PR #22155
CC @Wyverald