Closed elliefm closed 2 years ago
Wanted to make sure this works okay with older Cyrus releases... but it turns out the new Instance sanity checks in #146 can cause non-clean shutdown and cascading errors, and the sanity checks find problems on older Cyrus releases, so the tests all blow up dramatically.
So off down another rabbit hole I go...
Seems like this'll be okay once #154 is merged, so waiting for that.
This will let us treat 'httpmodules' et al as the set of flags it is, rather than an opaque string.
Need this for some refactoring I have planned for #151, but splitting it out so it can be reviewed separately