search

passiomatic / coldsweat

Web RSS aggregator and reader compatible with the Fever API
MIT License
145 stars 21 forks source link

ETag implementation no tested #55

Closed tewe closed 10 years ago

tewe commented 10 years ago

There is a copy/paste error in fetcher.py: modified_on instead of modified_since. Nobody noticed as it isn't used anywhere.

It looks like none of the various ETag occurences has tests.

passiomatic commented 10 years ago

Yep, I'm refactoring that portion of the fetcher code to implement #6 so I didn't bother to fix that.

passiomatic commented 10 years ago

I've refactored the code to always use new fetch_url function instead of that dumb check_url.