UnboundLocalError: local variable 'now' referenced before assignment

[Cyril N.]

Running the dkim.verify() code, I got the following exception on some cases:

  File "/var/www/venv/local/lib/python2.7/site-packages/dkim/__init__.py", line 1037, in verify │
    return d.verify(dnsfunc=dnsfunc) │
  File "/var/www/venv/local/lib/python2.7/site-packages/dkim/__init__.py", line 704, in verify │
    validate_signature_fields(sig) │
  File "/var/www/venv/local/lib/python2.7/site-packages/dkim/__init__.py", line 252, in validate_signature_fields │
    if x_sign < now - slop: │
UnboundLocalError: local variable 'now' referenced before assignment

[Scott Kitterman (kitterman)]

Thanks. I see the problem.

'now' is set when the t= Signature Timestamp signature tag is parsed and then used when the x= Signature Expiration tag is parsed and evaluated as well.

[Cyril N. (cnicodeme)]

Wow that was fast! Nice job! Thank you :)

[Scott Kitterman (kitterman)]

Fixed in 0.7.0.