[IOT-963] Fix maximum URI length 73/71973/1 accepted/tizen/common/20160603.150732 accepted/tizen/ivi/20160603.070512 accepted/tizen/mobile/20160603.070227 accepted/tizen/tv/20160603.070350 accepted/tizen/wearable/20160603.070421 submit/tizen/20160603.003854
authorInga Stotland <inga.stotland@intel.com>
Mon, 18 Apr 2016 22:41:52 +0000 (15:41 -0700)
committeryoungman <yman.jung@samsung.com>
Mon, 30 May 2016 04:42:26 +0000 (13:42 +0900)
According to core spec, maximum URI length could be up to 256

Change-Id: Ic9037fcca8a6ef49b0695ac5d3515f84aa11c773
Signed-off-by: Inga Stotland <inga.stotland@intel.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/7821
Tested-by: jenkins-iotivity <jenkins-iotivity@opendaylight.org>
Reviewed-by: Habib Virji <habib.virji@samsung.com>
resource/csdk/stack/include/ocstackconfig.h
resource/csdk/stack/test/stacktests.cpp

index 1bab0fc..8680c7f 100644 (file)
@@ -33,7 +33,7 @@
  * Maximum length of the URI supported by client/server while processing
  * REST requests/responses.
  */
-#define MAX_URI_LENGTH (64)
+#define MAX_URI_LENGTH (256)
 
 /**
  * Maximum length of the query supported by client/server while processing
index 037edcd..a5b9fe4 100644 (file)
@@ -453,7 +453,7 @@ TEST(StackResource, CreateResourceBadUri)
     OIC_LOG(INFO, TAG, "Starting CreateResourceBadUri test");
     InitStack(OC_SERVER);
 
-    const char *uri65 = "ABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKL";
+    const char *uri257 = "ABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVW";
 
     OCResourceHandle handle;
 
@@ -476,7 +476,7 @@ TEST(StackResource, CreateResourceBadUri)
     EXPECT_EQ(OC_STACK_INVALID_URI, OCCreateResource(&handle,
                                             "core.led",
                                             "core.rw",
-                                            uri65, //"/a/led",
+                                            uri257, //"/a/led",
                                             0,
                                             0,
                                             OC_DISCOVERABLE|OC_OBSERVABLE));