Default-Gateway Option about staticdhcpd

[GoogleCodeExporter]

Hi, Alexsandr.

This is actually intentional, as part of the resolution for issue #26.

I can still revert the token to 'gateway' to maintain compatibility, but I 
wanted to communicate that it is actually possible to supply multiple default 
gateways as part of option 3 (router).

I'll re-read the code and decide whether to stick with 'gateways' or 'gateway' 
in a few hours.

-Neil

Original comment by red.hamsterx on 7 Jul 2014 at 1:53

• Changed state: Accepted

[GoogleCodeExporter]

Oh, wait. I see the actual problem you're having now.

It's actually splitting the IP address into four addresses. That's not good and 
I'll get that fixed as soon as possible.

Original comment by red.hamsterx on 7 Jul 2014 at 3:49

[GoogleCodeExporter]

Thanks

Original comment by [email protected] on 7 Jul 2014 at 3:53

[GoogleCodeExporter]

r736 should correct this problem. For some strange reason, I hadn't considered 
the possibility that a Definition might be initialised with IPv4 objects as 
input.


After discussing the "gateway" -> "gateways" change from issue #26 with others, 
that will stay (so please update your httpdb.py file to include the 's'). 
Please let me know if you come across any places in the code where it was not 
changed, however.

-Neil

Original comment by red.hamsterx on 7 Jul 2014 at 4:34

[GoogleCodeExporter]

Sorry that write her. 

Update to latest svn ( svn checkout 
http://staticdhcpd.googlecode.com/svn/branches/2.0.0 staticdhcpd-read-only ), 
and get error on startup:
Traceback (most recent call last):
  File "./staticDHCPd", line 60, in <module>
    import staticdhcpdlib.config
  File "/usr/local/src/staticdhcpd-read-only/staticDHCPd/staticdhcpdlib/config.py", line 374, in <module>
    conf.extensions = _Namespace()
  File "/usr/local/src/staticdhcpd-read-only/staticDHCPd/staticdhcpdlib/config.py", line 315, in __init__
    self.__final = final
  File "/usr/local/src/staticdhcpd-read-only/staticDHCPd/staticdhcpdlib/config.py", line 328, in __setattr__
    raise AttributeError("Namespace does not support direct assignment")
AttributeError: Namespace does not support direct assignment

Change in 
/usr/local/src/staticdhcpd-read-only/staticDHCPd/staticdhcpdlib/config.py 
string 308 
from:
    __final = False #: If True, then no new layers will be created
to:
    __final = True #: If True, then no new layers will be created

server starting but error then process request get new error:

2014-07-07 21:53:59,029 : DEBUG : dhcp:178[Thread-8] : REQUEST:RENEW request 
from 00:0b:82:4e:16:0d processed in 0.0124 seconds
2014-07-07 21:53:59,031 : DEBUG : extension.httpdb:73[Thread-9] : MAC response 
received from 'http://exemple.serer/api/g-voip/dhcp/get/' for 
'00:0b:82:4e:16:0d'
2014-07-07 21:53:59,031 : ERROR : extension.httpdb:116[Thread-9] : Failed to 
lookup '00:0b:82:4e:16:0d' on 'http://exemple.serer/api/g-voip/dhcp/get/': 
'Definition' object has no attribute '_parse_address'
2014-07-07 21:53:59,031 : CRITICAL : dhcp:164[Thread-9] : Unable to handle 
REQUEST:RENEW from  00:0b:82:4e:16:0d:
Traceback (most recent call last):
  File "/usr/local/src/staticdhcpd-read-only/staticDHCPd/staticdhcpdlib/dhcp.py", line 388, in wrappedHandler
    f(self, wrapper)
  File "/usr/local/src/staticdhcpd-read-only/staticDHCPd/staticdhcpdlib/dhcp.py", line 576, in _handleDHCPRequest
    self._handleDHCPRequest_RENEW_REBIND(wrapper)
  File "/usr/local/src/staticdhcpd-read-only/staticDHCPd/staticdhcpdlib/dhcp.py", line 666, in _handleDHCPRequest_RENEW_REBIND
    definition = wrapper.retrieveDefinition()
  File "/usr/local/src/staticdhcpd-read-only/staticDHCPd/staticdhcpdlib/dhcp.py", line 369, in retrieveDefinition
    self._definition = self._server.getDatabase().lookupMAC(self.mac) or config.handleUnknownMAC(
  File "/etc/staticDHCPd/1/httpdb.py", line 129, in lookupMAC
    return self._lookupMAC(mac)
  File "/etc/staticDHCPd/1/httpdb.py", line 104, in _lookupMAC
    extra=result.get('extra'),
  File "/usr/local/src/staticdhcpd-read-only/staticDHCPd/staticdhcpdlib/databases/generic.py", line 95, in __init__
    self.ip = self._parse_address(ip)
AttributeError: 'Definition' object has no attribute '_parse_address'

Original comment by [email protected] on 7 Jul 2014 at 6:55

[GoogleCodeExporter]

The first one is because that feature was meant to be tested by someone else, 
as part of issue #22. I thought I had left it disabled for now; this has been 
fixed in r738.

The second is a lesson in why writing code on a phone is a bad idea: the 
indentation levels were wrong and the display was too small for me to notice. 
This is fixed in r739.

Original comment by red.hamsterx on 7 Jul 2014 at 8:23

[GoogleCodeExporter]

Big thanks again  

Original comment by [email protected] on 7 Jul 2014 at 8:41

[GoogleCodeExporter]

No, thank you for reporting these problems. :)

I should be done with documentation work for now, so I'll get a testing 
environment online shortly and try to make sure everything is stable for 2.0.0.

Original comment by red.hamsterx on 7 Jul 2014 at 10:00

[GoogleCodeExporter]

Original comment by red.hamsterx on 7 Jul 2014 at 10:00

• Changed state: Fixed