[IOT-2906][ES] Remove "href" from child response. 99/23399/2 1.3.1-RC6
authorSenthil Kumar G S <senthil.gs@samsung.com>
Wed, 22 Nov 2017 17:25:16 +0000 (22:55 +0530)
committerUze Choi <uzchoi@samsung.com>
Thu, 23 Nov 2017 08:15:44 +0000 (08:15 +0000)
For requests which come directly to child resources
(/WiFiConfResURI, /CoapCloudConfResURI and /DevConfResURI),
this patch will remove the "href" property from the response
as spec doesn't mandate the addition of "href" in the response.

And also, CTT expects the response from direct request to any child
to be same as the response which comes for the same child
through collection.

For example:
Properties in "rep" for '/WiFiConfResURI' should be same as
the properties in the direct response for GET from '/WiFiConfResURI' resource.

Currently "href" is not included in "rep" for child resources.
But it is included in the direct response for GET/POST.
When CTT compares both responses, it finds them different due of "href".

This patch resolves the issue and still conforms to spec.

Change-Id: I017bed18d3e0ad568b662a03d9bd80134538a8c0
Signed-off-by: Senthil Kumar G S <senthil.gs@samsung.com>
service/easy-setup/enrollee/src/easysetup.c
service/easy-setup/enrollee/src/resourcehandler.c

index 0bf3d73..5d88a13 100644 (file)
@@ -281,7 +281,7 @@ ESResult ESTerminateEnrollee()
         return ES_ERROR;
     }
 
-    OIC_LOG(ERROR, ES_ENROLLEE_TAG, "ESTerminateEnrollee success");
+    OIC_LOG(INFO, ES_ENROLLEE_TAG, "ESTerminateEnrollee success");
     return ES_OK;
 }
 
index cab9ac4..cad86ce 100644 (file)
@@ -854,12 +854,13 @@ OCRepPayload* constructResponseOfWiFiConf(char *interface, ES_BATCH_UPDATE_RESPO
     }
 
     OIC_LOG(DEBUG, ES_RH_TAG, "constructResponse WiFiConf res");
-    OCRepPayloadSetUri(payload, OC_RSRVD_ES_URI_WIFICONF);
 
     OCRepPayload* repPayload = NULL;
     OCRepPayload* tempPayload = NULL;
     if (!strcmp(interface, OC_RSRVD_INTERFACE_BATCH))
     {
+        OCRepPayloadSetUri(payload, OC_RSRVD_ES_URI_WIFICONF);
+
         repPayload = OCRepPayloadCreate();
         if (!repPayload)
         {
@@ -991,12 +992,13 @@ OCRepPayload* constructResponseOfCoapCloudConf(char *interface, ES_BATCH_UPDATE_
     }
 
     OIC_LOG(DEBUG, ES_RH_TAG, "constructResponse CoapCloudConf res");
-    OCRepPayloadSetUri(payload, OC_RSRVD_ES_URI_COAPCLOUDCONF);
 
     OCRepPayload* repPayload = NULL;
     OCRepPayload* tempPayload = NULL;
     if (!strcmp(interface, OC_RSRVD_INTERFACE_BATCH))
     {
+        OCRepPayloadSetUri(payload, OC_RSRVD_ES_URI_COAPCLOUDCONF);
+
         repPayload = OCRepPayloadCreate();
         if (!repPayload)
         {
@@ -1087,12 +1089,13 @@ OCRepPayload* constructResponseOfDevConf(char *interface, ES_BATCH_UPDATE_RESPON
     }
 
     OIC_LOG(DEBUG, ES_RH_TAG, "constructResponse DevConf res");
-    OCRepPayloadSetUri(payload, OC_RSRVD_ES_URI_DEVCONF);
 
     OCRepPayload* repPayload = NULL;
     OCRepPayload* tempPayload = NULL;
     if (!strcmp(interface, OC_RSRVD_INTERFACE_BATCH))
     {
+        OCRepPayloadSetUri(payload, OC_RSRVD_ES_URI_DEVCONF);
+
         repPayload = OCRepPayloadCreate();
         if (!repPayload)
         {