clicon / clixon-controller

Clixon network controller
Apache License 2.0
14 stars 5 forks source link

Using the characters '<' and '>' might cause an invalid diff #73

Closed krihal closed 9 months ago

krihal commented 10 months ago

In Junos I configured a new interface:

set interfaces test description "<*>"

This causes a diff even if there should be no diff:

debian@khn-dev[/]# commit diff
crpd1:
crpd1:
         <interface xmlns="http://yang.juniper.net/junos/conf/root">
            <name>test</name>
-          <description><*></description>
+          <description>></description>
         </interface>
crpd2:
OK
olofhagsand commented 10 months ago

I have some problems recreating this.

In Junos I configured a new interface: set interfaces test description "<*>"

Do you mean on the device itself, not via the controller?

olofhagsand commented 10 months ago

XML encoding missing, and extra hostnames printed. caveat: fixed in openconfig context not verified on junos as the original issue decribes.