refactor: use slices.Contains to simplify code

[bigmoonbit]

There is a new function added in the go1.21 standard library, which can make the code more concise and easy to read.

Inspired by #17403 and replace all

[bigmoonbit]

I have added the corresponding logic to ensure that when 'sub' is empty, it is treated as a wildcard.

Do I need to squash the two commits into one? Thank you for your review!

[taratorio]

I have added the corresponding logic to ensure that when 'sub' is empty, it is treated as a wildcard.

Do I need to squash the two commits into one? Thank you for your review!

no, all good - it will be squashed automatically when we merge it

[taratorio]

@bigmoonbit lint seems to be failing, can you please fix it and ill re-approve?

#18184 (comment)

[bigmoonbit]

@bigmoonbit lint seems to be failing, can you please fix it and ill re-approve?

@taratorio Modified. It seems to be because of an extra space.

Please review it again.

[taratorio]

@bigmoonbit lint seems to be failing, can you please fix it and ill re-approve?

@taratorio Modified. It seems to be because of an extra space.

Please review it again.

thank you