On 06/28/2011 02:08 PM, Rob Crittenden wrote:
Jan Cholasta wrote:
On 21.6.2011 14:15, Jan Cholasta wrote:
This patch adds a new option name_from_ip to dnszone commands. Default
value of idnsname is created from this option.

Honza


Fixed the API version number, added usage example to dns plugin help.

https://fedorahosted.org/freeipa/ticket/1045

Honza

Had quickie code review in IRC this morning. I asked for a comment around the while loop, Honza suggested: This is to make chained default_from work - idnssoarname default is created from idnsname and idnsname default is created from name_from_ip - without this change, idnssoarname default value isn't created when only name_from_ip is specified.

Would also be nice to have a test case for this new usage.

rob

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
NACK

Finally got the code to run, and I realize that it is completely a client side operation. That won't work for the WebUI.

The WebUI needs to use the same business logic as the CLI, but it cannot execute client side Python. Thus the API needs to accept the IP address, and calculate the reverse zone on it.

The reverse zone should honor the netmask. A discussion earlier today decided that if no netmask is specifified, use an assumed netmask of /64 for IPv6 and of /24 for IPv4.

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to