Added more audit logs

[EldoreiJK]

This should provide necessary logs to get some idea about the auditing results.
This is dependant on 42ity/fty-common-rest#36 and won't pass until it is merged and updated on Jenkins/Travis

[FrancoisRegisDegott-eaton]

Just a comment. On failure treatment, we invoke log_info_audit(). It should be log_error_audit().

[EldoreiJK]

Updated, thx

[jimklimov]

Missed some headers?

  CXX      src/shared/src_libfty_rest_la-magic.lo
ecppc --nolog -I./include -I./include -o ./src/web/src/add_gpio.cc ./src/web/src/add_gpio.ecpp && \
	    mv -f ./src/web/src/add_gpio.cc.cpp ./src/web/src/add_gpio.cc
  CXX      src/web/src/src_libfty_rest_la-add_gpio.lo
./src/web/src/add_gpio.ecpp: In member function 'virtual unsigned int {anonymous}::_component_::operator()(tnt::HttpRequest&, tnt::HttpReply&, tnt::QueryParams&)':
./src/web/src/add_gpio.ecpp:154:104: error: 'CHECK_USER_PERMISSIONS_OR_DIE_AUDIT' was not declared in this scope
     CHECK_USER_PERMISSIONS_OR_DIE_AUDIT (PERMISSIONS, audit_msg.empty () ? nullptr : audit_msg.c_str ());
                                                                                                        ^
make[1]: *** [src/web/src/src_libfty_rest_la-add_gpio.lo] Error 1
make[1]: Leaving directory `/home/travis/build/42ity/fty-rest'

Otherwise LGTM ;)

UPDATE: Missed at first the comment about dependency PRs

[jimklimov]

For future: macro names should be all same-style, commonly upper-cased.

Also, the CHECK_USER_PERMISSIONS_OR_DIE_AUDIT above casts (char *) nullptr while other cases I've seen recently do not...