faviconが存在するかチェックしているが現時点では無意味となっている
otofune opened this issue · comments
src/general.ts#L92-L94 でfaviconの存在が確認できたときのみというようにしているように見える
だが src/general.ts#L97 でnull時にチェックなしで URL.resolve(url.href, '/favicon.ico') を挿入してしまっている
ん？
icon ? って三項演算子使ってるからnullチェックしてることにならない？
いや、それはなるんだけど、
src/general.ts#L77-L95 で既に同じ URL.resolve(url.href, favicon) を存在するかチェックしてから挿入処理していて、挿入されていない (要するにファイルが実在しない) 時にもう1度3項演算子でnullチェックすることにより存在しないfaviconのURLを挿入してしまうんです
すいません、コード読み違えてました・・・このIssueのバグは誤報告です・・・
favicon周りで問題があるのはそうなんですが、問題が有るのはURL.resolve(url.href, '/favicon.ico') でした