Skip to content

Commit

Permalink
Do not crash on URIs without a host component.
Browse files Browse the repository at this point in the history
URIs such as `unix://...` (Unix domain socket) may legitimately be missing a
host component. We should gracefully handle them when looking for a credential
helper (no such helper can exist for them since the lookup is host-based, but
this could change in the future).

Fixes bazelbuild#16171 which is a regression in 5.3. We had some tests for this in
remote_execution_test.sh, but they were disabled due to flakiness in 8e3c0df.
I've filed bazelbuild#16185 to reenable them.

Closes bazelbuild#16184.

PiperOrigin-RevId: 470724658
Change-Id: Ibd7203546f00fe2335c14e7a5fa6d5bf64868bac
  • Loading branch information
tjgq authored and copybara-github committed Aug 29, 2022
1 parent e8278ed commit 0f18786
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@

import com.google.common.annotations.VisibleForTesting;
import com.google.common.base.Preconditions;
import com.google.common.base.Strings;
import com.google.common.collect.ImmutableMap;
import com.google.devtools.build.lib.vfs.Path;
import com.google.errorprone.annotations.Immutable;
Expand Down Expand Up @@ -66,7 +67,12 @@ private CredentialHelperProvider(
public Optional<CredentialHelper> findCredentialHelper(URI uri) {
Preconditions.checkNotNull(uri);

String host = Preconditions.checkNotNull(uri.getHost());
String host = uri.getHost();
if (Strings.isNullOrEmpty(host)) {
// Some URIs (e.g. unix://) legitimately have no host component.
return Optional.empty();
}

Optional<Path> credentialHelper =
findHostCredentialHelper(host)
.or(() -> findWildcardCredentialHelper(host))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,15 @@ public void invalidPattern() throws Exception {
assertInvalidPattern("foo-*.münchen.de");
}

@Test
public void uriWithoutHostComponent() throws Exception {
Path helper = fileSystem.getPath(EXAMPLE_COM_HELPER_PATH);
CredentialHelperProvider provider =
CredentialHelperProvider.builder().add("example.com", helper).build();

assertThat(provider.findCredentialHelper(URI.create("unix:///path/to/socket"))).isEmpty();
}

@Test
public void addNonExecutableDefaultHelper() throws Exception {
Path helper = fileSystem.getPath("/path/to/non/executable");
Expand Down

0 comments on commit 0f18786

Please sign in to comment.