aboutsummaryrefslogtreecommitdiff
path: root/src/main
diff options
context:
space:
mode:
authorAnton Tananaev <anton.tananaev@gmail.com>2020-01-23 08:49:30 -0800
committerAnton Tananaev <anton.tananaev@gmail.com>2020-01-23 08:49:30 -0800
commit1117d4abe4fe3545f9169b05269363c9655061b1 (patch)
treea8576320afc36f7342d9285d6e84ce755a087f35 /src/main
parenteb2697434dbbb8c12cad26df94e3a367880ac4db (diff)
downloadtrackermap-server-1117d4abe4fe3545f9169b05269363c9655061b1.tar.gz
trackermap-server-1117d4abe4fe3545f9169b05269363c9655061b1.tar.bz2
trackermap-server-1117d4abe4fe3545f9169b05269363c9655061b1.zip
Fix logging issue (fix #4473)
Diffstat (limited to 'src/main')
-rw-r--r--src/main/java/org/traccar/helper/ServletHelper.java26
1 files changed, 22 insertions, 4 deletions
diff --git a/src/main/java/org/traccar/helper/ServletHelper.java b/src/main/java/org/traccar/helper/ServletHelper.java
index e3481e249..b6c587ec3 100644
--- a/src/main/java/org/traccar/helper/ServletHelper.java
+++ b/src/main/java/org/traccar/helper/ServletHelper.java
@@ -1,8 +1,22 @@
+/*
+ * Copyright 2020 Anton Tananaev (anton@traccar.org)
+ *
+ * Licensed 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.
+ */
package org.traccar.helper;
import javax.servlet.http.HttpServletRequest;
-
public final class ServletHelper {
private ServletHelper() {
@@ -14,10 +28,14 @@ public final class ServletHelper {
String remoteAddress = request.getHeader("X-FORWARDED-FOR");
if (remoteAddress != null && !remoteAddress.isEmpty()) {
- return remoteAddress.substring(0, remoteAddress.indexOf(",")); // removes the additional data
+ int separatorIndex = remoteAddress.indexOf(",");
+ if (separatorIndex > 0) {
+ return remoteAddress.substring(0, separatorIndex); // remove the additional data
+ } else {
+ return remoteAddress;
+ }
} else {
- remoteAddress = request.getRemoteAddr();
- return remoteAddress;
+ return request.getRemoteAddr();
}
} else {
return null;