Make command line option for secure and observer

Implement command line option for secure resource and observer

Remove duplicate case to avoid confusion

Update app usage description

Change-Id: I2aa090a7a65448a7fbc54b41fef4b29d212f18f8
Signed-off-by: Sakthivel Samidurai <sakthivel.samidurai@intel.com>

Implemented Secure resource support in C++ layer

Added ability for the C++ stack to properly parse/support the secure protocol URIs.

Change-Id: I800d29598854339a2d30ae5b4ef89c2d0973dfe7
Signed-off-by: Sakthivel Samidurai <sakthivel.samidurai@intel.com>

Modified OCInitUDP to support security.

Added a new flag to OCInitUDP to indicated whether to reuse the address or not.
Modified create_new_context API. It now takes the ipAddr[4] as input. This needs
to be passed further down to dtls_init.

signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>
Change-Id: I536f215bf2a1ff4c90685e7a19e32fcb76f1e912

Implement sending host port number for secure resources

Secure resources are hosted on a separate IP port. All the
discovery responses are send from unicast unsecure port from
resource server. Currently, when a client receives discovery response
it 'assumes' that the sending port is the port hosting the resource
too. This 'assumption' is NOT valid for secure resources. Therefore,
there is a need to send 'secure port info' for secure resources in
JSON discovery response.

Change-Id: I54fd828a15913ec8be990d040e1190a88be43353
Signed-off-by: Sachin Agrawal <sachin.agrawal@intel.com>

Merge "Implemented libcoap's tinyDTLS interface"

Implemented libcoap's tinyDTLS interface

libcoap needs to encrypt/decrypt packets while sending on
secure port. This changeset implements interface for libcoap
to use tinyDTLS for creating DTLS sessions and performing
crypto operations.
This changeset is dependent on having tinyDTLS installed
under oic-resourc/resource/csdk directory.

Change-Id: I728b147991b94b802f125ad8e457c1830a114fba
Signed-off-by: Sachin Agrawal <sachin.agrawal@intel.com>

Merge "Added support for multicast presence"

Added C++ Sample application using multiple threads.

C++ Sample application demonstrating the idea of running multiple
servers and clients running in multiple threads.

Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>
Change-Id: Icdb59aaec35bb0c9d16b941fbf0eaa48c43d0a56

Merge "fix coding standard problem"

fix notification manager for m2

fix all code based on latest resource api.
fix notification folder name, following name convention.

Change-Id: Ic1eba69dd712428b4b0fae542d4acb005ad78a52

fix coding standard problem

fix all coding standard problem. This commit did NOT include logic changes .

Change-Id: I34f7a33374f137b8b641dff74e113b4941cb1391

fix things manager sdk for m2

Change-Id: I681378772f6b9dd120fd23abc124e66cef85c07e

Added support for multicast presence

Added OCMulticastNode data structure to store the nonce information of the server send
presence notifications. When a new notification comes to the client, client checks if there
is a multicast node, and then checks to see if the nonce is same or not. If same nonce, no call
back to client application is made. If nonce is different a call to client application is made.

Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>
Change-Id: Ia3fe163e7a304f499272f9ffa8a22a359f123d98

(1) add platform overload api : registerResource(resourcehandle, resource), (2) add host in rsrc_t

Change-Id: I4809566e153eb3f488617adb7e044da869c1d88c

Removed Iotivity logo.

Until we get clearance with Iotivity logo, we are not going to use any logo.

Signed off by: Sudarshan Prasad <sudarshan.prasad@intel.com>
Change-Id: I366fc53bf453c8c0ec4730b44f7f2418ae21a395

Repo Merge: Modified makefile for the soft sensor manager

Change-Id: I1f37e387d8c5d85b2ef95123126cc098e9b83735
Signed-off-by: William R. Dieter <william.r.dieter@intel.com>

Repo Merge: Modified makefiles for the things graph manager

Change-Id: I3c72a7b173c5070fb794ad59e2cb3866c53e9a9a
Signed-off-by: William R. Dieter <william.r.dieter@intel.com>

Repo Merge: Adding Arduino WiFi/Ethernet patches

Change-Id: I976cbec7819b89eaa72f106f5e82184bc3dc888a
Signed-off-by: William R. Dieter <william.r.dieter@intel.com>

Repo Merge: Move cJSON from oic-utilities to remove the dependency

Modified makefile for the cJSON move.

Change-Id: I48602f6c2bb8888690a1bf4a629df0c8981a8b3a
Signed-off-by: William R. Dieter <william.r.dieter@intel.com>

Repo Merge: Moving resource API down a directory

Change-Id: I92868cd59907eae66db7f18f7e9e1c65e02cc914
Signed-off-by: William R. Dieter <william.r.dieter@intel.com>

Updated Makefiles and CMakeLists.txt to point to resource, not oic-resource

Change-Id: I6a9b95042157d2f14ca8670a17198a487fc0fc3e
Signed-off-by: William R. Dieter <william.r.dieter@intel.com>

Added snapshot of oic-service repository

The original commit hash in oic-service is 6e5c509395d8fa54ea4849b7065c0e2d0bad6b0d.
No attempt is made in this commit to update Makefiles to build.  Makefile updates
will happen in a separate commit.

Change-Id: Ie541169ba2d10c10833c08b6ee90885f3cf704d0
Signed-off-by: William R. Dieter <william.r.dieter@intel.com>

Merge "fix darwin and ios build error"

fix darwin and ios build error

1) the logger implementation only consider arduino/android/linux
should add more platform in consider, such as darwin/ios ...

2) Without include <string>, there will be build error with xcode
compiler.

Change-Id: I6c98efdf40dc682f519faf90d04ae81ea406507b
Signed-off-by: Caiwen Zhang <caiwen.zhang@intel.com>

Merge "Updated libcoap to send messages using secure port"

Updated libcoap to send messages using secure port

Whenever a REST request is received by OIC Stack on secure
port, it needs to keep track of this fact for two reasons:
1) It allows upper layer to grant/deny access to secure resource
if the request was unauthenticated.
2) It also allows OC Stack to encrypt the response for a secure
request.
Since existing code was using a mechanism of send_flag to keep
track of various send mechanisms, it was modified to a bit mask
flag.

Change-Id: I293d4378a1e90234daf2553b6c6defdb6a3abac5
Signed-off-by: Sachin Agrawal <sachin.agrawal@intel.com>

Merge "Removed 'resource implementation' of libcoap library For enabling security in libcoap, there is a need to modify methods which are being invoked from 'resource interface' of libcoap. Since libcoap's 'resource interface' is unused in OIC stack, it does not make sense to spend dev cycles in keep on maintaining that code."

Removed 'resource implementation' of libcoap library
For enabling security in libcoap, there is a need to modify methods
which are being invoked from 'resource interface' of libcoap. Since
libcoap's 'resource interface' is unused in OIC stack, it does not
make sense to spend dev cycles in keep on maintaining that code.

Change-Id: I4a201b4bdc999fbe32ecfc051b24787741b51942
Signed-off-by: Sachin Agrawal <sachin.agrawal@intel.com>

Merge "Fix wrong return value from OCInitUDP API"

Fix wrong return value from OCInitUDP API

Call to setsocketopt method was hiding the return value from bind
method call (in case bind API fails). Modified code to use the
return value from bind call for API return value.

Change-Id: I7ab39b22444ba5db8a0b25524d4e5f9791c5ccf8
Signed-off-by: Sachin Agrawal <sachin.agrawal@intel.com>

Merge "Fixed bug where fridgeclient would immediately exit"

Merge "Modify C Sample Applications & solve memory leaks."

Merge "Fixed Iotivity logo."

Fixed Iotivity logo.

Since Iotivity is going public, the logo needs to match the correct logo.

Change-Id: I2e23f65553cf33228ed21fdde6a80bc80b9fa85a
Signed-off-by: Sudarshan Prasad <sudarshan.prasad@intel.com>

Merge "Fix crash issue after observation"

Fixed bug where fridgeclient would immediately exit

When removing the parameter from the constructor, I left the
parends in, which causes it to be interpreted as a function rather
than as an object.  This results in the condition_variable never being
hit.  This returns functionality to forever-blocking like it was
before.

Change-Id: I293f3260cc706db7e6cadde8515a1390bf1c283d
Signed-off-by: Erich Keane <erich.keane@intel.com>

Fix crash issue after observation

I discovered a crash in simpleserver C++ that when being debugged,
it would crash on the observation AFTER the client had cancelled.
The cause was that the NumberRecvHeaderOptions sent was an invalid
character rather than 0!  This fix repairs the issue by ensuring that
the values are initialized correctly.

Change-Id: I35a0a0aa45d356208f1cd563f3558997665881db
Signed-off-by: Erich Keane <erich.keane@intel.com>

Merge "Release notes for OCT 31, 2014 release"

Release notes for OCT 31, 2014 release

Updated release notes based on review comments.
Change-Id: I85e9cf672aaa9e2dbda73e27a7aea9ec51b0aa4a

Adding example of a Simple Attribute use on a collection resource using
C SDK.

This involved forming proper JSON examples, as well as adding test
cases to the application as the implementation of test cases were not
fullfilled yet.

Change-Id: I0b217a324334367bf3a923d7aadef5150e385821
Signed-off-by: Joseph Morrow <joseph.l.morrow@intel.com>

OCPlatform Object Removal

Removed dependency on the OCPlatform object.  We turned it into an internal-singleton(renamed to OCPlatform_impl so that we can use OCPlatform for the namespace, making it more consistent with previous code) with the expectation that people are to use OC::OCPlatform::<functionname> to control the API.
Change-Id: I3f0e13a27021f7454da68add93aaf562506e08ec

Modify C Sample Applications & solve memory leaks.

New Sample applications to demosntrate the basic functionality
of the stack. Added calls to free() to solve JSON related memory
leak issues.

Change-Id: I262fd7f1bca3d78fe46730632d46dc7abeb1d945
Signed-off-by: Omkar Hegde <omkar.m.hegde@intel.com>

Resolve name collision

Since these two files differed in their names only by case
it was not possible to check them both out at the same time
on a case insensitive filesystem.

Change-Id: I0ba508502b39bf47d157727ce7fb6501f58d68fb
Signed-off-by: David Warburton <david.warburton@intel.com>

Merge "roomserver/client updated to demonstrate simple attribute support in collections. Addressed review comments."

Merge "Adding C++ UB Unit Tests AND C++ UB Samples to buildScript.mk makefile."

Adding C++ UB Unit Tests AND C++ UB Samples to buildScript.mk makefile.

This change will be reflected by calling make on any 'Linux' "all" target
in the buildScript.mk makefile. This also contains refactor from UB & TB
to Rich & Lite, respectively.

Change-Id: Ie9c04c28c3f85bf5ffda29c6591cbd2eaeaac1e8

roomserver/client updated to demonstrate simple attribute support in collections.
Addressed review comments.

Change-Id: I4a054f1990954a9fd9dd3e98feddd204fd692676

Fixed server crash when we had less than MAX header options

Change-Id: I16a7a1c1049869bd2c26dcec903c2b3f82c11bfb

Added some string constants to cover exception cases
Note: I apparently messed up checking if this would compile/work :/  Build server is not working apparently.

Change-Id: I1b7eb33e4f9f1d184eb8c8488c0fe3052a1f1359

Added some string constants to cover exception cases

Change-Id: I102a7b136b6cc2d5c309e8c7984869aa51594e80

We wish to prevent multiple instances of OCPlatform from being accidentally created, so we make the copy-mechanisms private!
Patch 2: Fixed wording of comment in OCPlatform
Change-Id: Ic53816972426a29afc4764982a3ad712fa284945

Merge "Fixing the client.c in csdk/stack/test/linux to use OC_LOW_QOS instead of OC_NON_CONFIRMABLE. This is needed when we include unit tests in the build script."

Merge "Edited Arduino Makefiles and READMEs to describe how to ARDUINO_PORT."

Fixing many more regressions from String constants change.

Change-Id: Id1b6c0f5618c28720565e55354a73eade190f4a4

Edited Arduino Makefiles and READMEs to describe how to ARDUINO_PORT.

This fixes an issue that was stopping folks from installing to an Arduino
Due, but not Arduino Mega.

Patch 2: Fix mistype. I wasn't using the ARDUINO_PORT variable.

Change-Id: I8249077378a47cdedc721c5f3e8867c13661a183

Merge "Fixing a regression from the StringConstants change, we'd replaced the Property key with a different inadvertantly."

Fixing the client.c in csdk/stack/test/linux to use OC_LOW_QOS instead of OC_NON_CONFIRMABLE.
This is needed when we include unit tests in the build script.

Change-Id: Ib6effb0dd54a126f14e4efdef257693c69a767e2

Fixing a regression from the StringConstants change, we'd replaced the Property key with a different inadvertantly.

Change-Id: I8196194b860fc75c6f7d8f0bddebde5fdf379e87

Fix C++ Unittests build issue.

Change-Id: I82de00e976ebcf12b6f508e6fb6a222b6cd75e1f

Merge "Updated buildscript.mk to include build of TB unit tests and samples."

Merge "Patch 1: Unittests for C++ Stack: 1. Seperate directory for C++ unit tests. 2. tests.cpp file contains unit tests. 3. README explaining to create local.properties to indicate the location of gtest. 4. makefile to build unit tests with google testing framework. 5. One good sample unit test demonstrating the use of unit testing. Patch 2: Addressed review comments."

Merge "String Constant changes"

Merge "Reverted README back to instructions for buildScript.mk. As per Yamin, he simply acciddentally converted to something else.  He did not intend to push any changes to README.  I'm not sure how this change went through code review on Gerrit."

Updated buildscript.mk to include build of TB unit tests and samples.

I oops'd and some how overlooked this with the 'all_dev' target.

Patch 2: Swapping spots with parent. (ie. Switching parent-child dependency
between this and Change-Id: I1d20b09a095b320971765733b4343a7b781da1d9)

Patch 3: Bailing out on unit tests! The unit tests are rubbish. They
are wayyyyy out of date and WILL NOT build. THe unit test target is/was
pointing to some sample app by Jesse (that i presume was an accident
because it was put into the unit tests directory). This means we have NO
unit tests. This is bad.

Patch 4: Rebase onto master.

Change-Id: Ia9ca015a09bc3408c3ced99eb0432e0fb0814a12

Reverted README back to instructions for buildScript.mk. As per Yamin, he
simply acciddentally converted to something else.  He did not intend to
push any changes to README.  I'm not sure how this change went through
code review on Gerrit.

Change-Id: I3e9de7f5690fc129873f8e73eac1b1b78ba17fde

Patch 1:
Unittests for C++ Stack:
1. Seperate directory for C++ unit tests.
2. tests.cpp file contains unit tests.
3. README explaining to create local.properties to indicate the location of gtest.
4. makefile to build unit tests with google testing framework.
5. One good sample unit test demonstrating the use of unit testing.
Patch 2: Addressed review comments.

Change-Id: I6c0ec8af4f7fd13ce9bc8a56b7634074ac161b5f

String Constant changes

Change-Id: Ib90cc60fdd5c2ee44a697249af448e07116d75e8

Added commentary to the public API referring to new functionality of
notifications when a resource is deleted or when a server is shutdown
gracefully.

Patch 2: Edited commentary to OCStop() as per Erich's review.

Change-Id: I0b71bd3245072ac8f7cf9f67dff66b3d8b501355

Merge "Fix build issue for simpleserverHQ"

Merge "Release notes for Oct 14 release. Patch2: Incorporated review comments."

Fix build issue for simpleserverHQ

Change-Id: I54f2b45968559c0caa052bd53b1a900fd3178fe1

Release notes for Oct 14 release.
Patch2: Incorporated review comments.

Change-Id: Ieff471d87a8eeffe0e33768f9f70951d69e0adb7

1. C++ POST updates to include new URI and status for CoAP.
2. Now C++ Entity Handler also return OCEntityHandlerResult
3. Fix build issue.

Change-Id: I4c5e4ba7dab4079b3a45dfca62afc61ee11af22c

Merge "POST updates to include new URI and response status in CoAP header"

POST updates to include new URI and response status in CoAP header

This was previously submitted as review # 330 but ran into issues during rebase. Abandoning 330 and creating this one. Review comments in 330 have been addressed.

Change-Id: I3f39718975421a9e8e22e953623a81b9c7aac623

Added code for C++ implementation on Delete on both server and client ends.
Patch2 : Incorporated review comments
Change-Id: If666017890ec38890934bb01e53c9f89d49fee42

Merge "Delete old testapp files. They do not work and are obsolete."

Delete old testapp files. They do not work and are obsolete.

Change-Id: I1b4c1148acef98885b25a42ea3281aa126181948

Made small change to sample app to ensure that all clients are notified
when a server's resource is deleted.

Change-Id: I7eae7c365d9b570ac377d18e50c72e0477862d08

Client requests to delete a resource.
*Patch 1:
  1. Process request at C apps' level.
  2. Move logic into the C stack instead of app level.
  3. C stack responds to request to a 'to be deleted' resource.
*Patch 2: Added fixes per Sachin's code review.
*Patch 3: Added improvements per Jesse's code review.
***Abandoned changeset #293
***Restarted with changeset #304
*Patch 1: Rebased and changed Makefile per Jesse's feedback.
*Patch 2: Removed 'mark to be deleted' logic per design changes
          and added checks for entity handler result code.
*Patch 3: Fixed build error due to missing newly added coap code.
*Patch 4: Added improvements per Joey's feedbacks.
*Patch 5: Removed extra semicolon in enum to avoid build error.
*Patch 6: Rebased from Joey's changeset #313 and applied my changes on top.
*Patch 7: Rebased of master which has Joey's changeset #313, and added my changes on top.
*Patch 8: Added improvements per comments from Joey and Vijay

Change-Id: Iacec6f56c9f7ce7f61bfda8a47e84368edaf822e

Supporting QoS in C++ layer:
1) Modified the QoS enums so they are not transport specific
2) Overleaded member functions in Platform and Resource to allow specifying QoS
3) Modified internal function hierarchy to pass QoS to C layer
4) Added examples for C++: simpleclientHQ and simpleserverHQ to demo the usage of QoS
5) Modified examples to use the new QoS enums
6) Minor mods in the C layer
7) Minfor fix for OCICUC tests
8) Minfor fix for arduino sample
9) Fixes for received reviews
10)Rebase on to master
11)OCICUC : Platform bare pointer to reference

Change-Id: I5789935e16c851e7d47921a9acdfb15a363daa8e

Merge "Added logic to notify all observers when a resource has been deleted by OCDeleteResource API or when OCStop() is called. This will effectively tell all observers that "this" resource is no longer available. This information can be used by a client to infer that the server is going down."

Merge "Mods while working User Story US1742"

Added logic to notify all observers when a resource has been deleted by
OCDeleteResource API or when OCStop() is called. This will effectively tell
all observers that "this" resource is no longer available. This information
can be used by a client to infer that the server is going down.

If any more changes are req'q to encompass the new "Client able to delete
a resource" user story, a new changeset will be generated.

To come: A changeset containing a set (server and client) of examples
using this functionality.

Patch 2: Thuyen rebased on top of mine and for some reason needed to change
the author of my changeset, but doesn't appear to have made any changes to
my changeset other than rebasing it onto master.

Patch 3: Rebased on to newest master and made changes req' by Sachin and
Sudarshan in Patch 1 comments.

Change-Id: I343a5589c2988c45da95ba6d15685c21fa4704b6

Mods while working User Story US1742

1. Add additional stack and heap debug info to Arduino ocserver.cpp
2. Add optional debug info to ocmalloc.c.  Disabled by default.
3. Minor parameter name change to libcoap/mem.h
4. Mod libcoap/uthash.h to use OCMalloc instead of malloc

Rebased in response to broken build

Change-Id: I17e443f8dd1c0035140a1c0778532354e68900d8

Fix OCDoHandle comparation error

OCDoHanlde is 'void *', its size is uncertain. Compare the pointer
directly or must know its real size.

Change-Id: I91614fae4cee52637e9e0aeb42bfd150b63f34ee

Revert "Replaced listenCallback PTree with cereal"

This reverts commit 7e5b9d431a6ddf13c537c0e10bde0b7ecd866f19.

Change-Id: I3ccf81e13f0491394de58da7de4f7d64c439bbeb

Merge branch 'master' of ssh://oic/oic-resource

Replaced listenCallback PTree with cereal

Change-Id: I3ded4d42c495c73d5f2fb1d564722758ba8e5879

Merge "Fix but in OCUtility.cpp, exception information string wasn't being included."

Fix but in OCUtility.cpp, exception information string wasn't being included.

Change-Id: I89a20e6411220eb3be4604a8b8c12e5a6621c644

Merge "Fix type conflict"

Merge "Example to demonstrate the idea of sending json object as string in OCRepresenation"

Fix type conflict

In address.h, coap_address_t is defined as OCDevAddr, it should be used
as coap_address_t not 'struct coap_address_t'

Change-Id: I110ebc2e7064dad6588d4b3ae7625a2edde58093

Added clean up logic in makefile for ocicuc to remove new app
"small_example."

Change-Id: I564c7468582f449127ae54e96298cb7733b7ee6e

Example to demonstrate the idea of sending json object as string in OCRepresenation

Change-Id: I73a7c35241a55da856174223e121c3d808ed476f

Merge "fix for arduinodue "install" target for ocserver sample application."

Merge "Made additions to buildSCript and various makefiles to ensure a better "make clean" of all targets in the entire repo."

Merge "Tidy up logging call sites; throw in a few places. Remove old code (from my change). Clip lines. (ouch!)"

Made additions to buildSCript and various makefiles to ensure a better
"make clean" of all targets in the entire repo.

Patch 2: Cleaned up error that caused loop when "clean" target was called
by buildScipt.mk or makefile at root of repo.

Change-Id: Ib3b93a0a3a8b26c226ffdbf4851fa7176392a9b4