Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parameter position error: Regex.simpleMatch(dynamicSetting, setting.getKey()) in DynamicSettings.validateDynamicSetting() #7651

Closed
fangnoo opened this Issue Sep 9, 2014 · 1 comment

Comments

Projects
None yet
3 participants
@fangnoo
Copy link

fangnoo commented Sep 9, 2014

the right position is : Regex.simpleMatch( setting.getKey(), dynamicSetting)

@s1monw

This comment has been minimized.

Copy link
Contributor

s1monw commented Sep 9, 2014

thanks for opening this we will take care of it.

dakrone added a commit to dakrone/elasticsearch that referenced this issue Sep 9, 2014

Fix ordering of Regex.simpleMatch() parameters
Previously we incorrectly sent them in the wrong order, which can cause
validators not to be run for dynamic settings that have been added
matching a particular wildcard.

Fixes elastic#7651

@dakrone dakrone closed this in #7661 Sep 9, 2014

dakrone added a commit that referenced this issue Sep 9, 2014

Fix ordering of Regex.simpleMatch() parameters
Previously we incorrectly sent them in the wrong order, which can cause
validators not to be run for dynamic settings that have been added
matching a particular wildcard.

Fixes #7651
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.