Bug #5049
closed
Fix ValidatorConfig behavior with empty configuration
Added by Teng Liang about 5 years ago.
Updated over 3 years ago.
Description
The current ValidationPolicyConfig throws an exception when loading empty config section. A better behavior would be:
- allow loading an empty config file/section without throwing
- fail any validation attempts when the configuration is empty
Related issues
1 (1 open — 0 closed)
- Tracker changed from Task to Bug
- Category set to Security
- Subject changed from Improve validator behavior: empty config section can be loaded but fails validation to Fix ValidatorConfig behavior with empty configuration
- Description updated (diff)
Teng Liang wrote:
The current ValidationPolicyConfig throws an exception when loading empty config section, but then allows any validation.
I did not observe the second part of the behavior described above ("allows any validation"). @Teng, can you explain how to reproduce that?
- Status changed from New to Code review
- Assignee set to Davide Pesavento
- Target version set to v0.7
Teng Liang wrote:
Davide Pesavento wrote:
Teng Liang wrote:
The current ValidationPolicyConfig throws an exception when loading empty config section, but then allows any validation.
I did not observe the second part of the behavior described above ("allows any validation"). @Teng, can you explain how to reproduce that?
The test case in https://gerrit.named-data.net/c/NFD/+/5776/12/tests/daemon/mgmt/rib-manager-sl-announce.t.cpp#182, the validation should fail but succeed.
I still cannot reproduce (after applying the ndn-cxx patch of course, there's no way to test without it). Test cases behave as expected.
- Description updated (diff)
- Status changed from Code review to Closed
- % Done changed from 0 to 100
- Tags set to ValidatorConfig
Also available in: Atom
PDF
Updated by 
Updated by