Add gitreview file for Lucky Lamprey branch

Update the .gitreview file.

Change-Id: Ib55d20c3b6541239065610d1e61a3731a3f2727b
Bug-AGL: SPEC-4006
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Update .gitreview file

This updates the gitreview file in the project

.

Change-Id: Ie1fc9c696b4a1cf34cc184e2a42c026258cefe0a
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Message deprecating the submodule usage

Latest commit to the submodule which encourage the developpers to
use the separate library.

Bug-AGL: SPEC-1689

Change-Id: I5a8d539b881474fb17aabfc9034eb62ecd4f6a67
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

wrap-json: Improvements

* Most call "d = skip(++d);"  are replaced by
  "d = skip(d + 1);" that is equivalent but that
  virtually doesn't assign transiently d with d+1.

* make use of JSON_C_TO_STRING_NOSLASHESCAPE in
  test to improve output. Use a macro to reduce the
  code.

* enforce 'wrap_json_check' to use 'wrap_json_vcheck'
  and 'wrap_json_match' to use 'wrap_json_vmatch'.

Change-Id: I6e823da40415db4b54ec809455c93b39a8c5e9a5
Signed-off-by: José Bollo <jose.bollo@iot.bzh>

Revert "Added afb_list.h"

Reverting due to:
- Merged w/o full review cycle.
- License unclear.

This reverts commit a8b170a6131b6ad25e86745fdf10de63e16e3555.

Change-Id: I810dcca9da84d930716a29b2220e49a0859df3c5

Added afb_list.h

This is a user-space implementation of the kernel linked lists.
Massively used in the kernel, it is really light and
is contained in a single header.

Change-Id: I686b5fa5707b1c59339694c460ab8f0fec9679d4
Signed-off-by: Thierry Bultel <thierry.bultel@iot.bzh>

filescan-utils: follow symbolic links in search

Let the directory entities of symlink type be taken into
account.

Change-Id: Ic0197e81d1e0c761ca6a0fe9ea2dbcb56eee447b
Signed-off-by: Thierry Bultel <thierry.bultel@iot.bzh>

wrap-json: Fix duplicated lines in header file

Change-Id: I75558cf086b8b2d74297f9996abaa7f5a0b58cb7
Signed-off-by: José Bollo <jose.bollo@iot.bzh>

Add '@' as binder middle name separator.

Used binder name if set using the application id which use '@'
to separate name from version.

Change-Id: I1c4d963250ac17d4e4104b6cc69aa08b8fa95505
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Enhance 'wrap_json_clone_depth' description

Enhance 'wrap_json_clone_depth' description by adding details about
clone depth using this function.

Change-Id: Iede8fa6a697f12e8a32c1e15afdcef445edbcfe4
Signed-off-by: Jonathan Aillet <jonathan.aillet@iot.bzh>

Fix: use of GetBindingDirPath without dynapi

For classic binding < 3 without dynapi this fix the error
at compile time about conflicting type for this function.
This worked on eel and previous version because 'struct dynapi'
was declared and available for version 2 but now since binder
is in V3 then it isn't the case so function signature is not
valid anymore.

Change-Id: I135a723d21d70b8c54f4cab1c534210757318ed0
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Retrieve directory list from environment variables

This adds a function able to retrieve from a prefix and suffix
an environment variable. This is meant to search config or plugin
files from potentially several sources, environment and source code.

This tries to get 2 environments variables:

- prefix+binderName+suffix (eg: AFT_LOWCAN_CONFIG_PATH)
- prefix+suffix (eg: AFT_CONFIG_PATH)

Then it returns the one found or both if they exist with the
most accurate one (with the binder name) first followed by the
other.

Change-Id: Ic448ff017e6158bec05895d63688b8968b5c6434
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Remove the file .gitmodules

This project has no dependencency to any submodule.
Consequently it is cleaner to remove the file
.gitmodules

Change-Id: I097de9375a66c235b07626946ca8fcad196afbbd
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Remove unnecessary 'AFB_DEBUG' redefinition

Remove unnecessary 'AFB_DEBUG' redefinition when using app-fw v3.

Change-Id: I3b0271442495882e2a10762fe54b2f3f4ebe8e6b
Signed-off-by: Jonathan Aillet <jonathan.aillet@iot.bzh>

Use binding version to handle dynapi

Use binding version to define dynamic api debug messages function.
It is also cleaner to define binding version directly into
repository cmake than into a header like before this commit.

Change-Id: Ia14f635a7a2cec05c29f27ee4ce645711a8574a4
Signed-off-by: Jonathan Aillet <jonathan.aillet@iot.bzh>

At search, use the prefix parameter as a prefix

Be sure that prefix parameter is used as a prefix when searching for
configuration files (when using 'ScanForConfig' function).
Before this change, the function was only checking that the
prefix parameter was present in the file name (but not necessarily
at the beginning of the file name).

Bug-AGL:SPEC-1239

Change-Id: I9986cce5fab0fd5cd6d3049731f4c5883056a5f4
Signed-off-by: Jonathan Aillet <jonathan.aillet@iot.bzh>

Remove deprecated use of cmake/Qt macro

Last versions of cmake/Qt don't need the use of some macro anymore.
It allows you to put *.qrc files as sources files, which are
resources files. Those files need to be processed by a Qt's tool to
be transformed as C++ sources files.

Change-Id: Ide4b6eef53b3f6edfeac3c0553c79ce4277b1b7b
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>

Rename method to avoid conflict with Qt's one

Each QtObject define an 'event' method, which can conflict with the
'event' method to handle events in an Application Framework meaning.

Change-Id: I0fad844bfaa8fbfc2d14cc15b1f346b6f739a643
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>

Fix warnings due to json-c evolution

Library json-c now returns size_t instead of int.

Change-Id: Ib4f8bcd5dc6382fb3d189e83707a39dee8e14683
Signed-off-by: Sebastien Douheret <sebastien.douheret@iot.bzh>

wrap-json: new helper functions for comparison

The new functions are

 - wrap_json_clone_depth: clone with defined depth
 - wrap_json_sort: sort an array accordingly to wrap_json_cmp
 - wrap_json_keys: compute the array of sorted keys
 - wrap_json_cmp: compare 2 items
 - wrap_json_equal: test equallity
 - wrap_json_contains: test inclusion

Change-Id: I9787bff6c262fa2702c27802d771e2d487ce6309
Signed-off-by: jose bollo <jose.bollo@iot.bzh>

Removed anonymous function in ScanDir and fixed warnings

Change-Id: Ie6b67506be077f8fe5c2108e5dd98f9460dda485
Signed-off-by: Sebastien Douheret <sebastien.douheret@iot.bzh>

wrap-json: Add clone facility

This adds 3 new facilities:

 - wrap_json_clone: clones any json object superficially
 - wrap_json_clone_deep: clones any json object deeply
 - wrap_json_add: adds to an object the fields of an other object

Change-Id: I3844d972aa6477c9dde6f66ad0b0604284a853a7
Signed-off-by: José Bollo <jose.bollo@iot.bzh>

Disabled debug code (only used for testing).

Change-Id: I76e66e34f4dfe8d598d6ccf4c91e61932d5b75bc
Signed-off-by: Sebastien Douheret <sebastien.douheret@iot.bzh>

Correction to include dynamic api correctly

Define the correct version needed by the application framework
for dynamic api.

Change-Id: Ic0fd504316e2fc086545f0615ec6e83f2b1cae2d
Signed-off-by: Jonathan Aillet <jonathan.aillet@iot.bzh>

Fixed build warnings with gcc >= 7.3

Reduced the number of warnings in native host configuration (here
Tumbleweed) in order to not hide true/real warnings that may appear
in the future.

Signed-off-by: Sebastien Douheret <sebastien.douheret@iot.bzh>

Added afb-helpers-utils.h

Added this file to some useful macros, the first one here is INT_STR_MAX that
permits to get rid of a warning with gcc >= 7.2.0

Change-Id: I768d216ab5494d346d5ee6e0df66f93410bfab57
Signed-off-by: Thierry Bultel <thierry.bultel@iot.bzh>

Fixed compilation warnings with gcc-7.2.0

Replaced some 'size_t' by 'int'

Change-Id: I07492dad242b2c8f608f9cc90a4a257e48acec6a
Signed-off-by: Thierry Bultel <thierry.bultel@iot.bzh>

Fixed spelling

Signed-off-by: Sebastien Douheret <sebastien.douheret@iot.bzh>

More secure way to use strncat and strncpy

This ensure not data will be written out of bound
of the destination variable buffer

Change-Id: I4d736855f085fc5f728258c5928fed7191df46b4
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Fix: compile errors depending API version

Change-Id: I9c1476ab35f9bf70322ad4cd81ab0790e4d0724b
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

wrap-json: Fix warnings

Signed-off-by: jobol <jose.bollo@iot.bzh>

wrap-json: Update the documentation

Signed-off-by: jobol <jose.bollo@iot.bzh>

wrap-json: Accept null as valid base64

Signed-off-by: jobol <jose.bollo@iot.bzh>

wrap-json: Record result of tests

Signed-off-by: jobol <jose.bollo@iot.bzh>

wrap-json: Fix bug decoding empty base64

The function 'decode_base64' was buggy because
it freed 2 times the pointer 'result'.
This came from the fact realloc frees the
pointer and return NULL when the size if 0.

Signed-off-by: jobol <jose.bollo@iot.bzh>

wrap-json: Improve base64 en-de-coding

The optional characters '*' and '?' weren't managed
so the packing of y/Y base64 wasn't really entire.

Signed-off-by: jobol <jose.bollo@iot.bzh>

Fix: wrong searchPath parameter

Change initialization parts of the function to be clearer
and ensure that at least searchPath has been correctly passed

Change-Id: Ic055e2c244b261b33960072f51193fcbbd48c222
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Documentate, formating, linting.

Change-Id: Id77b9de1b847ca26133abd87511de4e78c7d01f7
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Adapt function to dynamic API

A dynamic API being dynamic, you need to pass as parameter
on behalf of which API you want to act.

If you are using classic v2 API then just pass a NULL
pointer value.

Change-Id: I9ac6b606ff25c63a05a647e418a7c2bcc4b2a7fe
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Formating

Change-Id: I16ee9154f4f5a7e71af83a22899a94f00e07aa79
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Make CURL Wrapping functions optional

Search CURL package in your system and add it
if found. This will add a libcurl to be linked
with the target.

Change-Id: Ie4a720fd11de97abead7a92a7fb18b5c8af21ec5
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Add new function to retrieve HTTP response code

Change-Id: I1d01c95d6418646ef9d2f8f4b07210a33b10c117
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

POST using unescaped arguments

Add the ability to make a POST request without
urlencode the arguments. This will only concatenate
them instead.
You can also specify the used separator.

Change-Id: Icb61a5a20771a89e89159132365d86ee4c982d1c
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Merge "Added a Qt's websocket client to AFB into helpers"

Added a Qt's websocket client to AFB into helpers

This add a single Qt class to serve as a client
to Application Framework Binder.
It's optional, not enabled by default.
To enable you have to set AFB_HELPERS_QTWSCLIENT
to ON.

Change-Id: Ia0759a95688e48183e6661082693c410a575b14b
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>

Fix GCC < v7 warning on maybe unitialized variable

Initialization done under a switch case statement within a "for"
loop which throw warning with GCC v6.

Change-Id: I47b51598cc68943bab6f98bd87b7ba6f40caeafc
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

wrap-json: Add base64 byte buffer support

Signed-off-by: jobol <jose.bollo@iot.bzh>

Rename cmake target accordingly to the git repo

Change-Id: I7e05205fe392884c25d1370564cad61119894754
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Adds a CURL wrapper library to help using libcurl

Change-Id: If73bab16a5d4a5258f730c599630bd5fa8e5684f
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Linting/Fix: remove warning int/size_t conversion

Change-Id: I831bac418f5c12c9a747b08ea8be72dcbe1d4ea3
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Fix memory leak

Change-Id: Ia02a639ea9210ed7961a88c91ff9e4cd1a732416
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Avoid memory collision.

Change-Id: I805a725d6b9477fcb690d7c2c7cbb6bb4f112dfb
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Add gitreview

Change-Id: I63e63cf1cc4545a4090a005629508a654e4a72d1
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Fix to support pre-V3 DynAPI

Prepare for future API-V3

Misc Fix/Cleaning

- Correct size type ssize_t to test
- Remove free() statement because memory don't issued from a malloc
causing exception
- Remove redundant break statement after a return.

Change-Id: I47824dd5cf7afe68cd32e1c3e94e69f31d802995
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Concatenate result directly from GetBindingDirPath

Return non const char pointer.

Change-Id: I975b18f72d26c1a2749ca54e4f096271497047dd
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Fix: mismatching function signature.

Change-Id: I5d3c8b34a4e42c867fe1e3eccd96aaa2cc9c5ca6
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Only return binding root dir.

If subdirs path is needed then concatenation as to be done
using the rootdir.

Change-Id: I50a1eeb1f853ae26995a96e1661ff38cb7959f60
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

More reliable GetBindingDirsPath function.

Only return a wanted directory path not a complex
structure.

Change-Id: I5b5a1dea2c55ee6a76dbbbd52c3ebfdd684255e5
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Adding JSON library for CPP and its docs

Change-Id: I3ae419ea586256bd9fff084f42966abf0f3491dd
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Retrieve binder rootdir and subfolder paths

Change-Id: I064bf4831be40de70dab68b72489c4fdfc69e9c4
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Reduce scope for some variables

Change-Id: Ic89421054a83709eec6afaa22dbb4373bcff9b19
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Include afb-utilities from C++ easily

Change-Id: Ic1c815b412d372a418c2faa51fea451c9100693c
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Rename back define variable.

Change-Id: Ie0d40f08230170ef780c46748889e153618d1c44
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

fix strncat funct

I: Statement might be overflowing a buffer in strncat. Common mistake:
  BAD: strncat(buffer,charptr,sizeof(buffer)) is wrong, it takes the left over size as 3rd argument
  GOOD: strncat(buffer,charptr,sizeof(buffer)-strlen(buffer)-1)

Signed-off-by: Ronan Le Martret <ronan.lemartret@iot.bzh>

MD linting

Change-Id: Iea7fb484aa13d4cba0e27bef06592218710f7c8a
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Fix: set default value if missing defined var

Change-Id: I93f8546e2cfe7fc11fdf0f956f10429f6c0ad8d0
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Format and missing header dependencies

Change-Id: If5890445bdb6a676ee6c435f50a27caa8ca5850d
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Remove audio-bindings dependencies

Change-Id: Iad5c51cca8a659ca3c77dd63ce7920e0f54faf63
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Initializing the repo

Change-Id: I354a1c136a57e1ec8d511041fbd13a19b730c419
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

Initial commit