XGraphKhipu / haarpcache

Fork of thundercache, haarpcache one smart cache.
perufw.wordpress.com
GNU General Public License v2.0
18 stars 18 forks source link

dont cache youtube dynamic video with id=o-* #5

Closed jagalk closed 11 years ago

jagalk commented 11 years ago

youtube back to dynamic id for new video example 1:

id=o-AO8OmUK1giAl_ilLbD5uJwrwG6OXp_hzob-3hsSai_dX id=o-AOziIEUJk0HN4NAhnTJ-Bg9VgASaZQVTDa-tun-DEdIa&ip

except old video static id, still cached : id=f1e46cfcee52d9c8 id=0a5ae5d93dd292b7

alfrenovsky commented 11 years ago

Both of your dynamic ID samples have 46 characters.

Youtube plugin allready discards ids longer than 40 chars

if (valor.at(0) == "id" || valor.at(0) == "video_id") { file = valor.at(1); if ( file.size() > 40 ) { // For videos with variables id. file = ""; return; } }

so stop caching videos with o-* ids is not necessary