Comment 4 for bug 1497424

Revision history for this message
Tim Burke (1-tim-z) wrote : Re: Date validation bypassed if 'Date' header missing

Patch seems good, and unit tests fail without the change to request.py. I wouldn't mind seeing a test with x-amz-date and no date header (in particular, I was surprised that test_date_header_with_x_amz_date_valid failed when I rolled back request.py), but it's not the sort of thing that should block this from landing.

Still need to make sure the func tests pass, though. Will get that set up tonight or tomorrow.