opendev/gerrit
Code Issues Proposed changes

Allow running without gerrit.canonicalWebUrl

Browse Source 
Gerrit's CanonicalWebUrl class is already responsible of getting
the full URL from either gerrit.config or the incoming HTTP request.

However, over time, other parts of Gerrit injected directly the
configured string through @CanonicalWebUrl annotation and did not
manage correctly the situation where the configuration is absent.

The only situation where gerrit.canonicalWebUrl is mandatory is
OAuth authentication, because the OAuth provider needs to have
a fixed callback URL with a single hostname.

Change-Id: Idee1d36eb08c53a0d06604b9a737695322e124d1
(cherry picked from commit 53eab1080b)
This commit is contained in:
Luca Milanesio
2017-07-14 17:21:04 +01:00
committed by Paladox none
parent 7bd3b9cfe2
commit a5420f0155
3 changed files with 14 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
gerrit-httpd/src/main/java/com/google/gerrit/httpd/raw/IndexServlet.java
View File

@@ -17,6 +17,7 @@ package com.google.gerrit.httpd.raw;
import static java.nio.charset.StandardCharsets.UTF_8; import static java.nio.charset.StandardCharsets.UTF_8;
import static javax.servlet.http.HttpServletResponse.SC_OK; import static javax.servlet.http.HttpServletResponse.SC_OK;
import com.google.common.base.Strings;
import com.google.common.io.Resources; import com.google.common.io.Resources;
import com.google.gerrit.common.Nullable; import com.google.gerrit.common.Nullable;
import com.google.template.soy.SoyFileSet; import com.google.template.soy.SoyFileSet;
@@ -61,6 +62,10 @@ public class IndexServlet extends HttpServlet {
} }
static String computeCanonicalPath(String canonicalURL) throws URISyntaxException { static String computeCanonicalPath(String canonicalURL) throws URISyntaxException {
if (Strings.isNullOrEmpty(canonicalURL)) {
return "";
}
// If we serving from a sub-directory rather than root, determine the path // If we serving from a sub-directory rather than root, determine the path
// from the cannonical web URL. // from the cannonical web URL.
URI uri = new URI(canonicalURL); URI uri = new URI(canonicalURL);

6
gerrit-server/src/main/java/com/google/gerrit/server/account/GetOAuthToken.java
View File

@@ -35,7 +35,7 @@ class GetOAuthToken implements RestReadView<AccountResource> {
private final Provider<CurrentUser> self; private final Provider<CurrentUser> self;
private final OAuthTokenCache tokenCache; private final OAuthTokenCache tokenCache;
private final String hostName; private final Provider<String> canonicalWebUrlProvider;
@Inject @Inject
GetOAuthToken( GetOAuthToken(
@@ -44,7 +44,7 @@ class GetOAuthToken implements RestReadView<AccountResource> {
@CanonicalWebUrl Provider<String> urlProvider) { @CanonicalWebUrl Provider<String> urlProvider) {
this.self = self; this.self = self;
this.tokenCache = tokenCache; this.tokenCache = tokenCache;
this.hostName = getHostName(urlProvider.get()); this.canonicalWebUrlProvider = urlProvider;
} }
@Override @Override
@@ -60,7 +60,7 @@ class GetOAuthToken implements RestReadView<AccountResource> {
} }
OAuthTokenInfo accessTokenInfo = new OAuthTokenInfo(); OAuthTokenInfo accessTokenInfo = new OAuthTokenInfo();
accessTokenInfo.username = a.getUserName(); accessTokenInfo.username = a.getUserName();
accessTokenInfo.resourceHost = hostName; accessTokenInfo.resourceHost = getHostName(canonicalWebUrlProvider.get());
accessTokenInfo.accessToken = accessToken.getToken(); accessTokenInfo.accessToken = accessToken.getToken();
accessTokenInfo.providerId = accessToken.getProviderId(); accessTokenInfo.providerId = accessToken.getProviderId();
accessTokenInfo.expiresAt = Long.toString(accessToken.getExpiresAt()); accessTokenInfo.expiresAt = Long.toString(accessToken.getExpiresAt());

7
gerrit-server/src/main/java/com/google/gerrit/server/plugins/PluginLoader.java
View File

@@ -629,9 +629,14 @@ public class PluginLoader implements LifecycleListener {
} }
private String getPluginCanonicalWebUrl(String name) { private String getPluginCanonicalWebUrl(String name) {
String canonicalWebUrl = urlProvider.get();
if (Strings.isNullOrEmpty(canonicalWebUrl)) {
return "/plugins/" + name;
}
String url = String url =
String.format( String.format(
"%s/plugins/%s/", CharMatcher.is('/').trimTrailingFrom(urlProvider.get()), name); "%s/plugins/%s/", CharMatcher.is('/').trimTrailingFrom(canonicalWebUrl), name);
return url; return url;
} }