search

gobwas / glob

Go glob
MIT License
948 stars 63 forks source link

Not correct matching with alternatives and separator #23

Closed valichek closed 6 years ago

valichek commented 6 years ago 
glob.MustCompile("{*.google.*,*.yandex.*}", '.').Match("www.google.com") // true, as expected
glob.MustCompile("{*.google.*,yandex.*}", '.').Match("www.google.com") // false, wrong!
gobwas commented 6 years ago

Hi! Thanks for the report and sorry for delay. I've found the bug inside the matching logic. It is not so trivial, so fixit it will take some time. Hope will finish it this week. Sorry.

gobwas commented 6 years ago

Hi @valichek fixed this for now. Sorry for such a long reaction. Thank you for the catch.