Skip to content

Commit

Permalink
[Fixes Azure#546] use lower case for Kudu url
Browse files Browse the repository at this point in the history
  • Loading branch information
jianghaolu committed Sep 10, 2018
1 parent 54fffd2 commit f7f8e9c
Show file tree
Hide file tree
Showing 4 changed files with 80 additions and 80 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,9 @@ class KuduClient {

KuduClient(WebAppBase webAppBase) {
service = webAppBase.manager().restClient().newBuilder()
.withBaseUrl("https://" + webAppBase.defaultHostName()
.withBaseUrl("https://" + webAppBase.defaultHostName().toLowerCase()
.replace("http://", "")
.replace(webAppBase.name(), webAppBase.name() + ".scm"))
.replace(webAppBase.name().toLowerCase(), webAppBase.name().toLowerCase() + ".scm"))
.withConnectionTimeout(3, TimeUnit.MINUTES)
.withReadTimeout(3, TimeUnit.MINUTES)
.build()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ public class WarDeployTests extends AppServiceTest {

@Override
protected void initializeClients(RestClient restClient, String defaultSubscription, String domain) {
WEBAPP_NAME = generateRandomResourceName("java-webapp-", 20);
WEBAPP_NAME = "JAVA" + generateRandomResourceName("webapp-", 20);

super.initializeClients(restClient, defaultSubscription, domain);
}
Expand Down
Loading

0 comments on commit f7f8e9c

Please sign in to comment.