search

syuilo / summaly

🔍 Get a summary of any web page
MIT License
47 stars 24 forks source link

faviconが存在するかチェックしているが現時点では無意味となっている #62

Closed otofune closed 7 years ago

otofune commented 7 years ago

src/general.ts#L92-L94 でfaviconの存在が確認できたときのみというようにしているように見える だが src/general.ts#L97 でnull時にチェックなしで URL.resolve(url.href, '/favicon.ico') を挿入してしまっている

syuilo commented 7 years ago

ん？

syuilo commented 7 years ago

icon ?って三項演算子使ってるからnullチェックしてることにならない？

otofune commented 7 years ago

いや、それはなるんだけど、 src/general.ts#L77-L95 で既に同じ URL.resolve(url.href, favicon) を存在するかチェックしてから挿入処理していて、挿入されていない (要するにファイルが実在しない) 時にもう1度3項演算子でnullチェックすることにより存在しないfaviconのURLを挿入してしまうんです

otofune commented 7 years ago

すいません、コード読み違えてました・・・このIssueのバグは誤報告です・・・ favicon周りで問題があるのはそうなんですが、問題が有るのはURL.resolve(url.href, '/favicon.ico') でした