eiri commented on issue #626: Don't include "decom" nodes in allowed_nodes URL: https://github.com/apache/couchdb/pull/626#issuecomment-312322215 Hmm, jiffy actually supporst JSON booleans ```erlang ([email protected])1> jiffy:decode(<<"{\"a\":true}">>). {[{<<"a">>,true}]} ([email protected])2> jiffy:decode(<<"{\"a\":\"true\"}">>). {[{<<"a">>,<<"true">>}]} ``` So this is more about invalid doc field allowed on admin /nodes endpoint rather than mem3 bug. I'd suggest either add validation on adding that "decom" field or make mem3 accept both boolean and string, after all you are trading one bug for another here. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
With regards, Apache Git Services
