Merge pull request #2312 from telefonicaid/fix/compare_gis_attribute_…

…type_enum enum.toString to equals compare
telefonicaid · Oct 10, 2023 · 9d18551 · 9d18551
2 parents bd11469 + 4558be0
commit 9d18551
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/CHANGES_NEXT_RELEASE b/CHANGES_NEXT_RELEASE
@@ -1,3 +1,4 @@
+- [cygnus-common] [Arcgis] fix url quoted based on uniqueFieldType (#2311)
 - [cygnus-common] [SQL] Add Primary Key on Timestamp to Error Log table (#2302)
 - [cygnus-common] Upgrade gson dependency from 2.6.2 to 2.10.1
 - [cygnus-common] Upgrade mysql-java-connector dependency from 8.0.28 to 8.0.33

diff --git a/...src/main/java/com/telefonica/iot/cygnus/backends/arcgis/restutils/ArcgisFeatureTable.java b/...src/main/java/com/telefonica/iot/cygnus/backends/arcgis/restutils/ArcgisFeatureTable.java
@@ -345,9 +345,9 @@ protected String getUniqueFieldList(List<Feature> featureArray, String uniqueFie
         // Checks if field exists in table, an retrieves it's type
         if (hasAttribute(uniqueField)) {
             String uniqueFieldType = getAttributeType(uniqueField);
-            quoted = GisAttributeType.DATE.equals(uniqueFieldType)
-                    || GisAttributeType.STRING.equals(uniqueFieldType);
-
+            quoted = GisAttributeType.DATE.toString().equalsIgnoreCase(uniqueFieldType)
+                || GisAttributeType.STRING.toString().equalsIgnoreCase(uniqueFieldType);
+            LOGGER.debug("uniqueField: " + uniqueField +  " uniqueFieldType " + uniqueFieldType + " in table");
             // Make the list
             for (Feature feature : featureArray) {
                 if (feature.getAttributes().containsKey(uniqueField)) {

diff --git a/cygnus-common/src/main/java/com/telefonica/iot/cygnus/backends/arcgis/restutils/RestApi.java b/cygnus-common/src/main/java/com/telefonica/iot/cygnus/backends/arcgis/restutils/RestApi.java
@@ -176,7 +176,9 @@ public static HttpResponse requestHTTP(String urlToRead, Map<String, String> par
 
         try {
             String strUrl = fullUrl(urlToRead, params);
-            LOGGER.debug("requesting HTTP: " + strUrl);
+            LOGGER.debug("requesting HTTP: " + strUrl +
+                         "---METHOD: " + httpMethod.toString() +
+                         "---BODY: " + body);
 
             // Create a trust manager that does not validate certificate chains
             TrustManager[] trustAllCerts = new TrustManager[] {new X509TrustManager() {