AdguardTeam / AdguardTeam/HostlistCompiler

Compress Not Working--Case Sensitivity.

未关闭
#59 0 条评论 0 个 reaction 已指派 1 人 已被 @maximtop 认领 在 GitHub 查看
Priority: P4
主要语言
JavaScript
星标
232
派生
30
PR 合并指标
30 天内没有已合并 PR

描述

I have my lists set to compress twice: once for each individual list so that my exclusions will work properly, and once at the end to compress fully the multiple lists that I use in order to remove redundant entries. The problem--or maybe it's a feature I overlooked--is that if a domain has an uppercase letter, HostlistCompiler doesn't acknowledge the redundant entries as redundant and leaves them in the list. For example, I have a list with the following entries after being compressed:
||2o7.net^
||ziffdavisetech.112.2O7.net^
||msstonojstemp.112.2O7.net^

Not sure why the second and third entries have a random capital O but I remember seeing this same thing occur in the Phishing Army list on Adguard DNS. There were two entries with the same name, but because one of them was capitalized, both entries were left in the list. There were a lot more 2o7.net entries but the compress transformation removed them, since ||2o7.net^ is in the list. Is it possible that these sites are supposed to be capitalized? I always thought that websites weren't case sensitive. Phishing Army has since fixed the entries in their list, so it doesn't happen anymore.

贡献指南

这个仓库没有索引到贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。