[ https://issues.apache.org/jira/browse/GEODE-3397?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16123599#comment-16123599 ]
ASF subversion and git services commented on GEODE-3397: -------------------------------------------------------- Commit 1bd15f8a27b9875d65910bd8ad51bd32122bc002 in geode's branch refs/heads/feature/GEODE-3416 from [~DivineEnder] [ https://git-wip-us.apache.org/repos/asf?p=geode.git;h=1bd15f8 ] GEODE-3397: Fixed issue with Tomcat locators in cache-client.xml file This closes #688 > Tomcat client server tests writes 2 locator tags to the cache-client.xml file > ----------------------------------------------------------------------------- > > Key: GEODE-3397 > URL: https://issues.apache.org/jira/browse/GEODE-3397 > Project: Geode > Issue Type: Bug > Components: http session > Reporter: David Anuta > Fix For: 1.3.0 > > > Client server tests for Tomcat 6, 7, and 8 that use Cargo write 2 locator > tags to the cache-client.xml file when it should only write one. Both the > default locator (on port 10334) and the locator that started up are written > to the xml file as seen below. > {noformat} > <?xml version="1.0" encoding="UTF-8" standalone="no"?><!-- > Licensed to the Apache Software Foundation (ASF) under one or more > contributor license agreements. See the NOTICE file distributed with > this work for additional information regarding copyright ownership. > The ASF licenses this file to You under the Apache License, Version 2.0 > (the "License"); you may not use this file except in compliance with > the License. You may obtain a copy of the License at > http://www.apache.org/licenses/LICENSE-2.0 > Unless required by applicable law or agreed to in writing, software > distributed under the License is distributed on an "AS IS" BASIS, > WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. > See the License for the specific language governing permissions and > limitations under the License. > --><client-cache xmlns="http://geode.apache.org/schema/cache" > xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" version="1.0" > xsi:schemaLocation="http://geode.apache.org/schema/cache > http://geode.apache.org/schema/cache/cache-1.0.xsd"> > <!-- The default pool connects to a locator running on localhost at > port 10334. To connect to a different locator host and port, modify > the following pool locator host and port. --> > <pool name="sessions" subscription-enabled="true"> > <locator host="localhost" port="10334"/> > <locator host="localhost" port="57621"/></pool> > <!-- To configure the client to use a server instead of a locator, replace > the locator pool above with the server pool below and modify the server > host and port as necessary. --> > <!-- > <pool name="sessions" subscription-enabled="true"> > <server host="localhost" port="40404"/> > </pool> > --> > > </client-cache> > {noformat} > This bug has to do with the editXMLFile function in the ContainerInstall > class. Specifically, it is caused by the TomcatContainer class not > overwriting the default locator tag, but instead creating a new one. This is > due to both a missing parameter in the TomcatContainer class when calling > editXMLFile, and due to the fact that the editXMLFile function does not have > a way to search for a tag by name instead of by id. > The editXMLFile function needs to be updated to allow for search by tag name > and the editOnSimilarAttributes parameter needs to be passed to the > editXMLFile function from the TomcatContainer class. -- This message was sent by Atlassian JIRA (v6.4.14#64029)