Easy Setup CoAPCloudConf Resource Update 89/24389/9
authorAbhishek Pandey <abhi.siso@samsung.com>
Wed, 14 Mar 2018 13:14:22 +0000 (18:44 +0530)
committerAbhishek Pandey <abhi.siso@samsung.com>
Thu, 15 Mar 2018 15:23:21 +0000 (15:23 +0000)
IOT-2963: https://jira.iotivity.org/browse/IOT-2963

In the upcoming version of OCF Spec, there are some changes in
definition of CoAPCloudConf Resource (oic.r.coapcloudconf). This patch
is for aligning the same with IoTivity implementation.

Following changes are done in this patch:

1) [Enrollee] Removed setting of Access Token Type (att) in GET
Response Payload.
2) [Enrollee] Removed reading of Access Token Type (att) from POST
Request Payload and passing a default value of "Bearer" to application.
3) [Enrollee] Removed Auth code Property from Enrollee. (this shouldn't
cause any harm).
4) [Mediator] Deprecated Getter and Setter APIs for Auth Code and
Access Token Type from Mediator

Change-Id: I6997b9e1b13cb9cd8bf95777e9984e4a749c87dd
Signed-off-by: Abhishek Pandey <abhi.siso@samsung.com>
16 files changed:
service/easy-setup/enrollee/inc/ESEnrolleeCommon.h
service/easy-setup/enrollee/src/resourcehandler.c
service/easy-setup/enrollee/src/resourcehandler.h
service/easy-setup/enrollee/unittests/ESEnrolleeTest.cpp
service/easy-setup/enrollee/unittests/ESMediatorSimulator.h
service/easy-setup/inc/escommon.h
service/easy-setup/mediator/richsdk/android/EasySetupCore/src/main/java/org/iotivity/service/easysetup/mediator/CloudProp.java
service/easy-setup/mediator/richsdk/android/EasySetupCore/src/main/java/org/iotivity/service/easysetup/mediator/ESConstants.java
service/easy-setup/mediator/richsdk/android/EasySetupCore/src/main/java/org/iotivity/service/easysetup/mediator/enums/OAUTH_TOKENTYPE.java
service/easy-setup/mediator/richsdk/inc/ESRichCommon.h
service/easy-setup/mediator/richsdk/src/RemoteEnrollee.cpp
service/easy-setup/mediator/richsdk/unittests/ESMediatorTest.cpp
service/easy-setup/sampleapp/enrollee/linux/enrolleewifi.c
service/easy-setup/sampleapp/enrollee/tizen-sdb/EnrolleeSample/enrolleewifi.cpp
service/easy-setup/sampleapp/mediator/linux/richsdk_sample/mediator.cpp
service/easy-setup/sampleapp/mediator/linux/richsdk_sample/submediator.cpp

index d33dff5..25548ba 100644 (file)
@@ -64,7 +64,6 @@ typedef struct
  */
 typedef struct
 {
-    char authCode[OIC_STRING_MAX_VALUE];        /**< Auth code issued by OAuth2.0-compatible account server **/
     char accessToken[OIC_STRING_MAX_VALUE];     /**< Access token resolved with an auth code **/
     OAUTH_TOKENTYPE accessTokenType;            /**< Access token type **/
     char authProvider[OIC_STRING_MAX_VALUE];    /**< Auth provider ID **/
index 2079ec6..d963843 100644 (file)
@@ -302,10 +302,8 @@ OCStackResult initCoapCloudConfResource(bool isSecured)
 {
     OCStackResult res = OC_STACK_ERROR;
 
-    OICStrcpy(g_ESCoapCloudConfResource.authCode, sizeof(g_ESCoapCloudConfResource.authCode), "");
     OICStrcpy(g_ESCoapCloudConfResource.accessToken, sizeof(g_ESCoapCloudConfResource.accessToken),
             "");
-    g_ESCoapCloudConfResource.accessTokenType = NONE_OAUTH_TOKENTYPE;
     OICStrcpy(g_ESCoapCloudConfResource.authProvider,
             sizeof(g_ESCoapCloudConfResource.authProvider), "");
     OICStrcpy(g_ESCoapCloudConfResource.ciServer, sizeof(g_ESCoapCloudConfResource.ciServer), "");
@@ -722,35 +720,24 @@ void updateCoapCloudConfResource(OCRepPayload* input)
         return;
     }
 
-    memset(cloudData->authCode, 0, OIC_STRING_MAX_VALUE);
     memset(cloudData->accessToken, 0, OIC_STRING_MAX_VALUE);
-    g_ESCoapCloudConfResource.accessTokenType = NONE_OAUTH_TOKENTYPE;
+    cloudData->accessTokenType = NONE_OAUTH_TOKENTYPE;
     memset(cloudData->authProvider, 0, OIC_STRING_MAX_VALUE);
     memset(cloudData->ciServer, 0, OIC_URI_STRING_MAX_VALUE);
     cloudData->userdata = NULL;
 
-    char *authCode = NULL;
-    if (OCRepPayloadGetPropString(input, OC_RSRVD_ES_AUTHCODE, &authCode))
-    {
-        OICStrcpy(g_ESCoapCloudConfResource.authCode, sizeof(g_ESCoapCloudConfResource.authCode), authCode);
-        OICStrcpy(cloudData->authCode, sizeof(cloudData->authCode), authCode);
-        OIC_LOG_V(INFO_PRIVATE, ES_RH_TAG, "g_ESCoapCloudConfResource.authCode %s", g_ESCoapCloudConfResource.authCode);
-    }
+
+    // As per OCF 1.4 Spec, Access Token Type property in not supported in CoAPCloudConf Resource.
+    // Hence, setting Token Type as "Bearer" by default.
+    g_ESCoapCloudConfResource.accessTokenType = OAUTH_TOKENTYPE_BEARER;
 
     char *accessToken = NULL;
     if (OCRepPayloadGetPropString(input, OC_RSRVD_ES_ACCESSTOKEN, &accessToken))
     {
         OICStrcpy(g_ESCoapCloudConfResource.accessToken, sizeof(g_ESCoapCloudConfResource.accessToken), accessToken);
         OICStrcpy(cloudData->accessToken, sizeof(cloudData->accessToken), accessToken);
-        OIC_LOG_V(INFO_PRIVATE, ES_RH_TAG, "g_ESCoapCloudConfResource.accessToken %s", g_ESCoapCloudConfResource.accessToken);
-    }
-
-    int64_t accessTokenType = -1;
-    if (OCRepPayloadGetPropInt(input, OC_RSRVD_ES_ACCESSTOKEN_TYPE, &accessTokenType))
-    {
-        g_ESCoapCloudConfResource.accessTokenType = accessTokenType;
         cloudData->accessTokenType = g_ESCoapCloudConfResource.accessTokenType;
-        OIC_LOG_V(INFO_PRIVATE, ES_RH_TAG, "g_ESCoapCloudConfResource.accessTokenType %d", g_ESCoapCloudConfResource.accessTokenType);
+        OIC_LOG_V(INFO_PRIVATE, ES_RH_TAG, "g_ESCoapCloudConfResource.accessToken %s", g_ESCoapCloudConfResource.accessToken);
     }
 
     char *authProvider = NULL;
@@ -774,7 +761,7 @@ void updateCoapCloudConfResource(OCRepPayload* input)
         gReadUserdataCb(input, OC_RSRVD_ES_RES_TYPE_COAPCLOUDCONF, &cloudData->userdata);
     }
 
-    if (authCode || accessToken || authProvider || ciServer)
+    if (accessToken || authProvider || ciServer)
     {
         OIC_LOG(DEBUG, ES_RH_TAG, "Send CoapCloudConfRsrc Callback To ES");
 
@@ -1058,9 +1045,7 @@ OCRepPayload* constructResponseOfCoapCloudConf(char *interface, ES_BATCH_UPDATE_
 
     if (RES_INCLUDE == resp)
     {
-        OCRepPayloadSetPropString(payload, OC_RSRVD_ES_AUTHCODE, g_ESCoapCloudConfResource.authCode);
         OCRepPayloadSetPropString(payload, OC_RSRVD_ES_ACCESSTOKEN, g_ESCoapCloudConfResource.accessToken);
-        OCRepPayloadSetPropInt(payload, OC_RSRVD_ES_ACCESSTOKEN_TYPE, (int)g_ESCoapCloudConfResource.accessTokenType);
         OCRepPayloadSetPropString(payload, OC_RSRVD_ES_AUTHPROVIDER, g_ESCoapCloudConfResource.authProvider);
         OCRepPayloadSetPropString(payload, OC_RSRVD_ES_CISERVER, g_ESCoapCloudConfResource.ciServer);
 
index 24728f8..6f3fef0 100644 (file)
@@ -72,7 +72,6 @@ typedef struct
 typedef struct
 {
     OCResourceHandle handle;
-    char authCode[OIC_STRING_MAX_VALUE];
     char accessToken[OIC_STRING_MAX_VALUE];
     OAUTH_TOKENTYPE accessTokenType;
     char authProvider[OIC_STRING_MAX_VALUE];
index a2e55e8..86b75b5 100644 (file)
@@ -448,7 +448,7 @@ TEST_F(EasysetupEnrolleeTest, CloudServerProperiesProvisionedWithSuccess)
     mocks.OnCallFunc(CoapCloudConfCbInApp).Do(
         [& cntForReceivedCallbackWithSuccess](ESCoapCloudConfData *data)
         {
-            if(!strcmp(data->authCode, "authCode") &&
+            if(!strcmp(data->accessToken, "accessToken") &&
                 !strcmp(data->authProvider, "authProvider") &&
                 !strcmp(data->ciServer, "ciServer"))
             {
index 3ef00ca..aa001eb 100644 (file)
@@ -456,7 +456,7 @@ private:
             if(m_remoteEnrollee != NULL)
             {
                 CloudProp cloudProp;
-                cloudProp.setCloudProp("authCode", "authProvider", "ciServer");
+                cloudProp.setCloudPropWithAccessToken("accessToken", "authProvider", "ciServer");
 
                 m_remoteEnrollee->provisionCloudProperties(cloudProp,
                     std::bind(&ESMediatorSimulator::cloudProvisioningStatusCallback,
index 8a52d83..7a24588 100644 (file)
@@ -57,6 +57,8 @@ extern "C"
 #define OC_RSRVD_ES_CRED                   "cd"
 #define OC_RSRVD_ES_AUTHTYPE               "wat"
 #define OC_RSRVD_ES_ENCTYPE                "wet"
+
+// Auth Code is no longer supported as per OCF 1.4 CoAPCloudConf Resource schema.
 #define OC_RSRVD_ES_AUTHCODE               "ac"
 #define OC_RSRVD_ES_ACCESSTOKEN            "at"
 #define OC_RSRVD_ES_ACCESSTOKEN_TYPE       "att"
index 044221b..b7ced52 100644 (file)
@@ -44,6 +44,11 @@ public class CloudProp {
         mCloudID = "";
     }
 
+    /**
+     * @deprecated OCF 1.4 Resource schema for CoAPCloudConf Resource no longer supports Auth Code
+     * property.
+     */
+    @Deprecated
     public void setCloudProp(String authCode, String authProvider, String ciServer)
     {
         if(authCode == null)
@@ -67,6 +72,12 @@ public class CloudProp {
         }
     }
 
+    /**
+     * @deprecated OCF 1.4 Resource schema for CoAPCloudConf Resource no longer supports Access
+     * Token Type property. Recommended to use overloaded method which doesn't expect tokenType
+     * input parameter instead.
+     */
+    @Deprecated
     public void setCloudPropWithAccessToken(String accessToken, OAUTH_TOKENTYPE tokenType,
                                         String authProvider, String ciServer)
     {
@@ -92,19 +103,56 @@ public class CloudProp {
         }
     }
 
+    /**
+     * This method sets Cloud Properties like access token, auth provider name and Cloud Server URL.
+     */
+    public void setCloudPropWithAccessToken(String accessToken,String authProvider, String ciServer)
+    {
+        if(accessToken == null)
+        {
+            accessToken = "";
+        }
+        if(authProvider == null)
+        {
+            authProvider = "";
+        }
+        if(ciServer == null)
+        {
+            ciServer = "";
+        }
+        try {
+            mRep.setValue(ESConstants.OC_RSRVD_ES_ACCESSTOKEN, accessToken);
+            mRep.setValue(ESConstants.OC_RSRVD_ES_AUTHPROVIDER, authProvider);
+            mRep.setValue(ESConstants.OC_RSRVD_ES_CISERVER, ciServer);
+        } catch (OcException e) {
+            Log.e(TAG, "setCloudPropWithAccessToken is failed: "  + e.toString());
+        }
+    }
+
+    /**
+     * This method sets Cloud Interface server's UUID.
+     */
     public void setCloudID(String cloudID)
     {
         mCloudID = cloudID;
     }
 
+    /**
+     * This method sets Cloud Interface server's credential ID of certificate.
+     */
     public void setCredID(int credID)
     {
         mCredID = credID;
     }
+
     /**
-     * This method returns the authCode used for the first registration to IoTivity cloud
-     * @return AuthCode for sign-up to IoTivity cloud
+     * This method returns the authCode used for the first registration to IoTivity Cloud.
+     *
+     * @return AuthCode for sign-up to IoTivity Cloud.
+     * @deprecated OCF 1.4 Resource schema for CoAPCloudConf Resource no longer supports Auth Code
+     * property.
      */
+    @Deprecated
     public String getAuthCode()
     {
         if(mRep == null)
@@ -147,7 +195,7 @@ public class CloudProp {
         return null;
     }
 
-       /**
+    /**
      * This method returns the Cloud Interface server's URL to be registered
      * @return CI server's URL to be registered
      */
@@ -189,8 +237,9 @@ public class CloudProp {
     }
 
     /**
-     * This method returns an accessToken used for the first registration to IoTivity cloud
-     * @return accessToken for sign-up to IoTivity cloud
+     * This method returns an accessToken used for the first registration to IoTivity Cloud.
+     *
+     * @return accessToken for sign-up to IoTivity Cloud.
      */
     public String getAccessToken()
     {
@@ -212,9 +261,12 @@ public class CloudProp {
     }
 
     /**
-     * This method returns an access token type
-     * @return tokenType of access token
+     * This method returns an access token type.
+     * @return Type of access token
+     * @deprecated OCF 1.4 Resource schema for CoAPCloudConf Resource no longer supports Access
+     * Token Type property. A default value of "Bearer" can be assumed.
      */
+    @Deprecated
     public OAUTH_TOKENTYPE getAccessTokenType()
     {
         if(mRep == null)
@@ -231,7 +283,9 @@ public class CloudProp {
         {
             Log.e(TAG, "getAccessTokenType is failed: "  + e.toString());
         }
-        return OAUTH_TOKENTYPE.NONE_OAUTH_TOKENTYPE;
+
+        /* Default Value for Access Token Type is "Bearer" */
+        return OAUTH_TOKENTYPE.OAUTH_TOKENTYPE_BEARER;
     }
 
     public OcRepresentation toOCRepresentation()
index f0088d6..6cfba4f 100644 (file)
@@ -35,6 +35,7 @@ public class ESConstants {
     public static final String OC_RSRVD_ES_CRED = "cd";
     public static final String OC_RSRVD_ES_AUTHTYPE = "wat";
     public static final String OC_RSRVD_ES_ENCTYPE = "wet";
+    // Auth Code is no longer supported as per OCF 1.4 CoAPCloudConf Resource schema.
     public static final String OC_RSRVD_ES_AUTHCODE = "ac";
     public static final String OC_RSRVD_ES_ACCESSTOKEN = "at";
     public static final String OC_RSRVD_ES_ACCESSTOKEN_TYPE = "att";
index 70dc519..bd15a32 100644 (file)
 package org.iotivity.service.easysetup.mediator.enums;
 
 /**
- * This enum class indicates an OAuth Token type like "bearer" and "mac"\r
+ * This enum class indicates an OAuth Token type like "bearer" and "mac".
+ *
+ * @deprecated OCF 1.4 Resource schema for CoAPCloudConf Resource no longer supports Access
+ * Token Type property. A default value of "Bearer" token type can be assumed.
  */
-public enum OAUTH_TOKENTYPE\r
+@Deprecated
+public enum OAUTH_TOKENTYPE
 {
-    NONE_OAUTH_TOKENTYPE(0),\r
-    OAUTH_TOKENTYPE_BEARER(1),\r
-    OAUTH_TOKENTYPE_MAC(2);\r
-\r
+    NONE_OAUTH_TOKENTYPE(0),
+    OAUTH_TOKENTYPE_BEARER(1),
+    OAUTH_TOKENTYPE_MAC(2);
+
     private int value;
 
-    private OAUTH_TOKENTYPE(int value)\r
+    private OAUTH_TOKENTYPE(int value)
     {
         this.value = value;
     }
 
     /**
-     * Get OAuth Token type as an integer value\r
+     * Get OAuth Token type as an integer value
      *
-     * @return int OAuth Token type as an integer value\r
+     * @return int OAuth Token type as an integer value
      */
     public int getValue()
     {
@@ -49,13 +53,13 @@ public enum OAUTH_TOKENTYPE
     }
 
     /**
-     * Get OAuth Token type as OAUTH_TOKENTYPE type value\r
+     * Get OAuth Token type as OAUTH_TOKENTYPE type value
      *
-     * @return OAuth Token type enum value corresponding to its integer value\r
+     * @return OAuth Token type enum value corresponding to its integer value
      */
-    public static OAUTH_TOKENTYPE fromInt(int i)\r
+    public static OAUTH_TOKENTYPE fromInt(int i)
     {
-        for (OAUTH_TOKENTYPE b : OAUTH_TOKENTYPE.values())\r
+        for (OAUTH_TOKENTYPE b : OAUTH_TOKENTYPE.values())
         {
             if (b.getValue() == i)
                 return b;
@@ -63,4 +67,4 @@ public enum OAUTH_TOKENTYPE
         return null;
     }
 }
-\r
+
index 6623f69..22a65f7 100644 (file)
@@ -166,8 +166,15 @@ namespace OIC
              * @param authCode  Auth code issued by OAuth2.0-compatible account server
              * @param authProvider Auth provider ID
              * @param ciServer Cloud interface server URL which an Enrollee is going to registered
+             * @deprecated OCF 1.4 Resource schema for CoAPCloudConf Resource no longer supports
+             * auth code property. Recommended to use access token for cloud configuration instead.
+             *
+             * @see setCloudPropWithAccessToken
              */
-            void setCloudProp(string authCode, string authProvider, string ciServer)
+            OC_DEPRECATED_MSG(
+            void setCloudProp(string authCode, string authProvider, string ciServer),
+                                        "Auth code is no longer supported",
+                                        2018.03)
             {
                 m_rep.setValue(OC_RSRVD_ES_AUTHCODE, authCode);
                 m_rep.setValue(OC_RSRVD_ES_AUTHPROVIDER, authProvider);
@@ -182,11 +189,17 @@ namespace OIC
              * @param tokenType Access token type, i.e. "bearer"
              * @param authProvider Auth provider ID
              * @param ciServer Cloud interface server URL which an Enrollee is going to registered
+             * @deprecated OCF 1.4 Resource schema for CoAPCloudConf Resource no longer supports
+             * Access Token Type property. Recommended to use overloaded function which doesn't expect
+             * tokenType input parameter instead.
              *
              * @see OAUTH_TOKENTYPE
              */
+            OC_DEPRECATED_MSG(
             void setCloudPropWithAccessToken(string accessToken, OAUTH_TOKENTYPE tokenType,
-                                                string authProvider, string ciServer)
+                                                string authProvider, string ciServer),
+                                        "Setting Access token type is no longer supported",
+                                        2018.03)
             {
                 m_rep.setValue(OC_RSRVD_ES_ACCESSTOKEN, accessToken);
                 m_rep.setValue(OC_RSRVD_ES_ACCESSTOKEN_TYPE, tokenType);
@@ -194,6 +207,22 @@ namespace OIC
                 m_rep.setValue(OC_RSRVD_ES_CISERVER, ciServer);
             }
 
+            /**
+             * Set CoapCloudConf resource properties with Access token to be delivered to Enrollee
+             *
+             * @param accessToken  Access token which is given in a return of auth code issued by
+             *                     OAuth2.0-compatible account server
+             * @param authProvider Auth provider ID
+             * @param ciServer Cloud interface server URL which an Enrollee is going to registered
+             */
+            void setCloudPropWithAccessToken(string accessToken, string authProvider,
+                                                string ciServer)
+            {
+                m_rep.setValue(OC_RSRVD_ES_ACCESSTOKEN, accessToken);
+                m_rep.setValue(OC_RSRVD_ES_AUTHPROVIDER, authProvider);
+                m_rep.setValue(OC_RSRVD_ES_CISERVER, ciServer);
+            }
+
             /**
              * Set CloudServer's UUID
              *
@@ -218,8 +247,13 @@ namespace OIC
              * Get an auth code to be delivered.
              *
              * @return an auth code to be delivered.
+             * @deprecated OCF 1.4 Resource schema for CoAPCloudConf Resource no longer supports
+             * auth code.
              */
-            std::string getAuthCode() const
+            OC_DEPRECATED_MSG(
+            std::string getAuthCode() const,
+                    "Auth code is no longer supported",
+                    2018.03)
             {
                 if(m_rep.hasAttribute(OC_RSRVD_ES_AUTHCODE))
                 {
@@ -294,16 +328,22 @@ namespace OIC
              * Get an access token type to be delivered.
              *
              * @return an access token type to be delivered.
+             * @deprecated OCF 1.4 Resource schema for CoAPCloudConf Resource no longer supports
+             * access token type.
              */
-            OAUTH_TOKENTYPE getAccessTokenType() const
+            OC_DEPRECATED_MSG(
+            OAUTH_TOKENTYPE getAccessTokenType() const,
+                            "Access token type is no longer supported",
+                            2018.03)
             {
-
                 if(m_rep.hasAttribute(OC_RSRVD_ES_ACCESSTOKEN_TYPE))
                 {
                     return static_cast<OAUTH_TOKENTYPE>(
                                 m_rep.getValue<int>(OC_RSRVD_ES_ACCESSTOKEN_TYPE));
                 }
-                return NONE_OAUTH_TOKENTYPE;
+
+                // Default value for Access Token Type is "Bearer"
+                return OAUTH_TOKENTYPE_BEARER;
             }
 
             /**
@@ -332,28 +372,19 @@ namespace OIC
                 }
 
                 // Representation should be updated as per OCF 1.3 Easy Setup resource schemas.
-                std::string authCode = m_rep.getValue<std::string>(OC_RSRVD_ES_AUTHCODE);
                 std::string authProvider = m_rep.getValue<std::string>(OC_RSRVD_ES_AUTHPROVIDER);
                 std::string ciServer = m_rep.getValue<std::string>(OC_RSRVD_ES_CISERVER);
 
                 std::string accessToken = m_rep.getValue<std::string>(OC_RSRVD_ES_ACCESSTOKEN);
-                OAUTH_TOKENTYPE accessTokenType = static_cast<OAUTH_TOKENTYPE>(
-                                m_rep.getValue<int>(OC_RSRVD_ES_ACCESSTOKEN_TYPE));
 
                 OCRepresentation rootRep;
                 rootRep.setUri(OC_RSRVD_ES_URI_EASYSETUP);
 
                 OCRepresentation cloudDataRep;
 
-                if(authCode.length() > 0)
-                {
-                    cloudDataRep.setValue(OC_RSRVD_ES_AUTHCODE, authCode);
-                }
-
                 if(accessToken.length() > 0)
                 {
                     cloudDataRep.setValue(OC_RSRVD_ES_ACCESSTOKEN, accessToken);
-                    cloudDataRep.setValue(OC_RSRVD_ES_ACCESSTOKEN_TYPE, accessTokenType);
                 }
 
                 cloudDataRep.setValue(OC_RSRVD_ES_AUTHPROVIDER, authProvider);
index 552f326..504f1c4 100644 (file)
@@ -549,7 +549,7 @@ namespace OIC
 
             m_cloudPropProvStatusCb = callback;
 
-            if((cloudProp.getAuthCode().empty() && cloudProp.getAccessToken().empty()) ||
+            if(cloudProp.getAccessToken().empty() ||
                 cloudProp.getAuthProvider().empty() ||
                 cloudProp.getCiServer().empty())
             {
index ee61731..00224d9 100644 (file)
@@ -464,7 +464,7 @@ TEST_F(ProvisionCloudPropertiesTest,
           ThrowExceptionWhenProvisionCloudPropertiesFailedByCallbackIsNull)
 {
     CloudProp cloudProp;
-    cloudProp.setCloudProp("authCode", "authProvider", "ciServer");
+    cloudProp.setCloudPropWithAccessToken("accessToken", "authProvider", "ciServer");
     cloudProp.setCloudID("f002ae8b-c42c-40d3-8b8d-1927c17bd1b3");
 
     EXPECT_ANY_THROW(g_remoteEnrollee->provisionCloudProperties(cloudProp, nullptr));
@@ -474,7 +474,7 @@ TEST_F(ProvisionCloudPropertiesTest,
           ThrowExceptionWhenProvisionCloudPropertiesFailedWithoutAuthCode)
 {
     CloudProp cloudProp;
-    cloudProp.setCloudProp("", "authProvider", "ciServer");
+    cloudProp.setCloudPropWithAccessToken("", "authProvider", "ciServer");
     cloudProp.setCloudID("f002ae8b-c42c-40d3-8b8d-1927c17bd1b3");
     EXPECT_ANY_THROW(g_remoteEnrollee->provisionCloudProperties(cloudProp,
                                                                 cloudPropProvStatusCb));
@@ -484,7 +484,7 @@ TEST_F(ProvisionCloudPropertiesTest,
           ThrowExceptionWhenProvisionCloudPropertiesFailedWithoutAuthProvider)
 {
     CloudProp cloudProp;
-    cloudProp.setCloudProp("authCode", "", "ciServer");
+    cloudProp.setCloudPropWithAccessToken("accessToken", "", "ciServer");
     cloudProp.setCloudID("f002ae8b-c42c-40d3-8b8d-1927c17bd1b3");
     EXPECT_ANY_THROW(g_remoteEnrollee->provisionCloudProperties(cloudProp,
                                                                 cloudPropProvStatusCb));
@@ -494,7 +494,7 @@ TEST_F(ProvisionCloudPropertiesTest,
           ThrowExceptionWhenProvisionCloudPropertiesFailedWithoutCIServer)
 {
     CloudProp cloudProp;
-    cloudProp.setCloudProp("authCode", "authProvider", "");
+    cloudProp.setCloudPropWithAccessToken("accessToken", "authProvider", "");
     cloudProp.setCloudID("f002ae8b-c42c-40d3-8b8d-1927c17bd1b3");
     EXPECT_ANY_THROW(g_remoteEnrollee->provisionCloudProperties(cloudProp,
                                                                 cloudPropProvStatusCb));
@@ -503,7 +503,7 @@ TEST_F(ProvisionCloudPropertiesTest,
 TEST_F(ProvisionCloudPropertiesTest, ProvisionCloudPropertiesSucceed)
 {
     CloudProp cloudProp;
-    cloudProp.setCloudProp("authCode", "authProvider", "ciServer");
+    cloudProp.setCloudPropWithAccessToken("accessToken", "authProvider", "ciServer");
     cloudProp.setCloudID("f002ae8b-c42c-40d3-8b8d-1927c17bd1b3");
 
     int cntForReceivedCallbackWithSuccess = 0;
@@ -528,7 +528,7 @@ TEST_F(ProvisionCloudPropertiesTest, ProvisionCloudPropertiesSucceed)
 TEST_F(ProvisionCloudPropertiesTest, ProvisionCloudPropertiesWithResourceSucceed)
 {
     CloudProp cloudProp;
-    cloudProp.setCloudProp("authCode", "authProvider", "ciServer");
+    cloudProp.setCloudPropWithAccessToken("accessToken", "authProvider", "ciServer");
     cloudProp.setCloudID("f002ae8b-c42c-40d3-8b8d-1927c17bd1b3");
 
     int cntForReceivedCallbackWithSuccess = 0;
index 70e8c95..3a8786b 100644 (file)
@@ -131,7 +131,6 @@ void CoapCloudConfProvCbInApp(ESCoapCloudConfData *eventData)
         return ;
     }
 
-    printf("AuthCode : %s\n", eventData->authCode);
     printf("AcessToken : %s\n", eventData->accessToken);
     printf("AcessTokenType : %d\n", eventData->accessTokenType);
     printf("AuthProvider : %s\n", eventData->authProvider);
index 8dc9437..3e38c73 100644 (file)
@@ -151,7 +151,6 @@ void CoapCloudConfProvCbInApp(ESCoapCloudConfData *eventData)
         return ;
     }
 
-    cout << "AuthCode : " << eventData->authCode << endl;
     cout << "AcessToken : " <<  eventData->accessToken << endl;
     cout << "AcessTokenType : "<< eventData->accessTokenType << endl;
     cout << "AuthProvider : " << eventData->authProvider << endl;
index 82b28c1..9cae990 100644 (file)
@@ -323,8 +323,7 @@ void provisionCloudProperty()
     }
 
     CloudProp cloudProp;
-    cloudProp.setCloudPropWithAccessToken("accessToken", OAUTH_TOKENTYPE_BEARER, "authProvider",
-                                          "ciServer");
+    cloudProp.setCloudPropWithAccessToken("accessToken", "authProvider", "ciServer");
     cloudProp.setCloudID("f002ae8b-c42c-40d3-8b8d-1927c17bd1b3");
     cloudProp.setCredID(1);
 
index 1e78f64..c2efc94 100644 (file)
@@ -323,8 +323,7 @@ void provisionCloudProperty()
     }
 
     CloudProp cloudProp;
-    cloudProp.setCloudPropWithAccessToken("accessToken", OAUTH_TOKENTYPE_BEARER, "authProvider",
-                                          "ciServer");
+    cloudProp.setCloudPropWithAccessToken("accessToken", "authProvider", "ciServer");
 
     try
     {