Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

range: failing validation does not set error message #1428

Closed
BernhardDenner opened this issue Mar 21, 2017 · 3 comments
Closed

range: failing validation does not set error message #1428

BernhardDenner opened this issue Mar 21, 2017 · 3 comments
Labels

Comments

@BernhardDenner
Copy link
Collaborator

I'm using the range plugin with a single range (didn't try multiple ranges). When trying to set an invalid value for the desired key, the range plugin fails but does not set an error message on the parent key.

e.g:

$> kdb set user/test/puppet-val/spec/x1 90
The command kdb set failed while accessing the key database with the info:

$> 

Seems to be an easy fix, since the error message is only set if the checking function return -1 but actually range fails return 0

@tom-wa
Copy link
Contributor

tom-wa commented Mar 21, 2017

thanks for reporting, #1429 fixes that issue

@markus2330
Copy link
Contributor

Thank you for reporting!, I merged #1429
@BernhardDenner Can you please close the issue if it is fixed?

@BernhardDenner
Copy link
Collaborator Author

fixed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants