search

shawn1m / overture

A customized DNS relay server
MIT License
1.79k stars 284 forks source link

fix suffix tree #239

Closed wzv5 closed 3 years ago

wzv5 commented 3 years ago

issue: When insert abc.com and 1.abc.com, abc.com will not be matched.

Add an final flag to the tree to solve this problem.

codecov[bot] commented 3 years ago

Codecov Report

Merging #239 into master will increase coverage by 6.12%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #239      +/-   ##
==========================================
+ Coverage   49.84%   55.97%   +6.12%     
==========================================
  Files          13       13              
  Lines         660      661       +1     
==========================================
+ Hits          329      370      +41     
+ Misses        283      241      -42     
- Partials       48       50       +2
Impacted Files Coverage Δ
core/matcher/suffix/tree.go 87.23% <100.00%> (+87.23%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update d5c127b...516c5fc. Read the comment docs.

shawn1m commented 3 years ago

Merged. Thanks!