Cache: get_offline_cachekey uses too little information from TextNode for hashing about django-compressor HOT 3 CLOSED

Here's an example of what goes into get_hexdigest in the current implementation:

['<Text Node: \'\n\t<link rel="stylesheet" \'>',
 '<Variable Node: STATIC_URL>',
 '<Text Node: \'sass/print.scss" type="te\'>']

from django-compressor.

I ran into this, it can be quite problematic depending on how your HTML is written. The proposed solution works for me, I wonder if it would be better to just detect that it's a TextNode and use render() instead of the internal "s" attribute though. Or something similar. Thoughts ?

from django-compressor.

I applied pilt's fix (without my previous suggestion, which I now believe was a bad idea :) in this fork liberation@58b7204 and modified the tests to fail without the patch, and pass with it :-)

from django-compressor.