Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#13188 closed Bug Report - Crash (fixed)

Datetime calculation error

Reported by: morten@… Owned by: Mark Spieth
Priority: minor Milestone: 29.1
Component: Bindings - Python Version: v29.0
Severity: medium Keywords:
Cc: Ticket locked: no

Description

In requests_cache_compatability.py line 40 it fails: 

self.cache.remove_old_entries(datetime.utcnow() - self._cache_expire_after)

as it aught to be: 

self.cache.remove_old_entries(datetime.utcnow() - timedelta(self._cache_expire_after))

Attachments (1)

expire_after.patch​ (813 bytes ) - added by Mark Spieth 8 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 by morten@…, 8 years ago

Original error message: 

requests_cache_compatability.py:40:remove_expired_responses:TypeError: unsupported operand type(s) for -: 'datetime.datetime' and 'int'

Pull request to solve problem: ​https://github.com/MythTV/mythtv/pull/162

comment:2 by Stuart Auchterlonie, 8 years ago

Milestone: unknown → 29.1

lgtm; :shipit:

comment:3 by Peter Bennett, 8 years ago

Owner: changed from Raymond Wagner to Mark Spieth
Status: new → assigned

Assigning to Mark as he did the update to ttvdb code.

comment:4 by Mark Spieth, 8 years ago

The appended solution is not sufficient for all versions of requests_cache. It needs to check if its not a timedelta and then convert it if necessary to a timedelta with seconds= as the param. I have a patch ready but just need to test before I commit.

by Mark Spieth, 8 years ago

Attachment: expire_after.patch​ added

comment:5 by Mark Spieth <mspieth@…>, 8 years ago

Resolution: → fixed
Status: assigned → closed

In 68e982ee82f079c92d7ccd007859fff36541a4ed/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:6 by Mark Spieth <mspieth@…>, 8 years ago

In 152529781bdc01f26974f4dc24ddae9202740849/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:7 by Mark Spieth <mspieth@…>, 8 years ago

In 5d9b8f1979c84325110cfca881cee9b15f5cabc0/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available
Note: See TracTickets for help on using tickets.