Saving Changes in DHCPv6 Relay fails
-
Saving any changes under this service generates a 'Notices' bell, showing the configuration has been reverted..
pfSenseConfigurator
pfSense is restoring the configuration /cf/conf/backup/config-1451265285.xml
-
I have been unable to reproduce this.
Would you provide more details of your firewall and configuration please? what are the exact steps to reproduce?
-
Steps to reproduce seem simple. Try to enable the IPv6 relay service, any interface, any destination server.
This is the configuration section added by services_dhcpv6_relay.php
<dhcrelay6><enable><interface>lan</interface>
<server><0>IPV6 IP Address</server></enable></dhcrelay6>It turns out there is an error in the system log:
Dec 30 14:30:08 php-fpm /services_dhcpv6_relay.php: New alert found: pfSense is restoring the configuration /cf/conf/backup/config-1451419652.xml
Dec 30 14:30:08 php-fpm /services_dhcpv6_relay.php: pfSense is restoring the configuration /cf/conf/backup/config-1451419652.xml
Dec 30 14:30:08 php-fpm /services_dhcpv6_relay.php: XML error: XML_ERR_NAME_REQUIRED at line 5246 in /conf/config.xmlWhich by the line number mentioned, seems to indicate that it doesn't like the numeral tags
This seems like it should be fairly easy to duplicate…... Is there any further information that would be useful?
-
That helps. Thanks.
-
Changing config section to read as follows, resolves XML issue:
<dhcrelay6><enable><interface>lan</interface>
<server>IPV6 IP Address</server></enable></dhcrelay6> -
Looks like the server list should be a comma separated list.
I have fixed this in both services_dhcp_relay.php and services_dhcpv6_relay.php
The "Append circuit ID and agent ID to requests" checkboxes were broken too :(
Would you mind testing this please? My abilities to test with V6 is limited.
Thanks.
-
This all looks correct now.
-
This mostly works now :)
The remaining (new?) problem is that the Destination Address field is blank upon returning to the page of the service, when enabled. This happens for both IPv4 and 6 Relay services.
-
This piece had list and array confused as too :(
Just pushed a fix for that.
-
Fixed.
Thank you :D