experiments/clixon
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

misc documentation

Browse source
This commit is contained in:
Olof hagsand 2020-07-28 14:00:09 +02:00
parent 23bcee8185
commit 3ecc259ec7
4 changed files with 22 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

6
CHANGELOG.md
View file

@ -42,6 +42,10 @@ Expected: July 2020
### API changes on existing protocol/config features (For users) ### API changes on existing protocol/config features (For users)
* Netconf lock/unlock behaviour changed to adhere to RFC 6241
* Changed commit lock error tag from "lock denied" to "in-use".
* Changed unlock error message from "lock is already held" to #lock not active" or "lock held by other session".
* Fixed [lock candidate succeeded even though it is modified #110](https://github.com/clicon/clixon/issues/110)
* New clixon-config@2020-06-17.yang revision * New clixon-config@2020-06-17.yang revision
* Added CLICON_CLI_LINES_DEFAULT for setting window row size of raw terminals * Added CLICON_CLI_LINES_DEFAULT for setting window row size of raw terminals
* Added enum HIDE to CLICON_CLI_GENMODEL for auto-cli * Added enum HIDE to CLICON_CLI_GENMODEL for auto-cli
@ -49,7 +53,6 @@ Expected: July 2020
* CLICON_SSL_SERVER_CERT * CLICON_SSL_SERVER_CERT
* CLICON_SSL_SERVER_KEY * CLICON_SSL_SERVER_KEY
* CLICON_SSL_CA_CERT * CLICON_SSL_CA_CERT
* Restconf FCGI (eg via nginx) have changed reply message syntax slightly as follows (due to refactoring and common code with evhtp): * Restconf FCGI (eg via nginx) have changed reply message syntax slightly as follows (due to refactoring and common code with evhtp):
* Bodies in error retuns including html code have been removed * Bodies in error retuns including html code have been removed
* Some (extra) CRLF:s have been removed * Some (extra) CRLF:s have been removed
@ -95,6 +98,7 @@ Expected: July 2020
### Corrected Bugs ### Corrected Bugs
* Fixed: Don't call upgrade callbacks if no revision defined so there's no way to determine right way 'from' and 'to' * Fixed: Don't call upgrade callbacks if no revision defined so there's no way to determine right way 'from' and 'to'
* Fixed: [lock candidate succeeded even though it is modified #110](https://github.com/clicon/clixon/issues/110)
* Fixed: [Need to add the possibility to use anchors around patterns #51](https://github.com/clicon/cligen/issues/51): * Fixed: [Need to add the possibility to use anchors around patterns #51](https://github.com/clicon/cligen/issues/51):
* Dont escape `$` if it is last in a regexp in translation from XML to POSIX. * Dont escape `$` if it is last in a regexp in translation from XML to POSIX.
* Fixed `CLICON_YANG_UNKNOWN_ANYDATA` for config and state data. This feature introduced in 4.5 didnt really work. * Fixed `CLICON_YANG_UNKNOWN_ANYDATA` for config and state data. This feature introduced in 4.5 didnt really work.

22
lib/src/clixon_string.c
View file

@ -61,16 +61,16 @@
* by the caller * by the caller
* *
* @code * @code
* char **vec = NULL; * char **vec = NULL;
* char *v; * char *v;
* int nvec; * int nvec;
* if ((vec = clicon_strsep("/home/user/src/clixon", "/", &nvec)) == NULL) * if ((vec = clicon_strsep("/home/user/src/clixon", "/", &nvec)) == NULL)
* err; * err;
* for (i=0; i<nvec; i++){ * for (i=0; i<nvec; i++){
* v = vec[i]; * v = vec[i];
* ... * ...
* } * }
* free(vec); * free(vec);
* @endcode * @endcode
* @param[in] string String to be split * @param[in] string String to be split
* @param[in] delim String of delimiter characters * @param[in] delim String of delimiter characters
@ -538,7 +538,6 @@ xml_chardata_cbuf_append(cbuf *cb,
return retval; return retval;
} }
/*! Split a string into a cligen variable vector using 1st and 2nd delimiter /*! Split a string into a cligen variable vector using 1st and 2nd delimiter
* Split a string first into elements delimited by delim1, then into * Split a string first into elements delimited by delim1, then into
* pairs delimited by delim2. * pairs delimited by delim2.
@ -729,7 +728,6 @@ clicon_str2int_search(const map_str2int *mstab,
return -1; /* not found */ return -1; /* not found */
} }
/*! Split colon-separated node identifier into prefix and name /*! Split colon-separated node identifier into prefix and name
* @param[in] node-id * @param[in] node-id
* @param[out] prefix If non-NULL, return malloced string, or NULL. * @param[out] prefix If non-NULL, return malloced string, or NULL.

2
lib/src/clixon_yang_parse_lib.c
View file

@ -1014,7 +1014,7 @@ yang_parse_post(clicon_handle h,
if (yang_cardinality(h, yspec->ys_stmt[i], yang_argument_get(yspec->ys_stmt[i])) < 0) if (yang_cardinality(h, yspec->ys_stmt[i], yang_argument_get(yspec->ys_stmt[i])) < 0)
goto done; goto done;
/* 3: Check features: check if enabled and remove disabled features */ /* 3: Check features/if-features: check if enabled and remove disabled features */
for (i=modnr; i<yang_len_get(yspec); i++) for (i=modnr; i<yang_len_get(yspec); i++)
if (yang_features(h, yspec->ys_stmt[i]) < 0) if (yang_features(h, yspec->ys_stmt[i]) < 0)
goto done; goto done;

7
yang/clixon/clixon-config@2020-06-17.yang
View file

@ -336,7 +336,12 @@ module clixon-config {
description description
"Treat unknown XML/JSON nodes as anydata when loading from startup db. "Treat unknown XML/JSON nodes as anydata when loading from startup db.
This does not apply to namespaces, which means a top-level node: xxx:yyy This does not apply to namespaces, which means a top-level node: xxx:yyy
is accepted only if yyy is unknown, not xxx"; is accepted only if yyy is unknown, not xxx.
Note that this option has several caveats which needs to be fixed. Please
use with care.
The primary issue is that the unknown->anydata handling is not restricted to
only loading from startup but may occur in other circumstances as well. This
means that sanity checks of erroneous XML/JSON may not be properly signalled.";
} }
leaf CLICON_BACKEND_DIR { leaf CLICON_BACKEND_DIR {
type string; type string;