At the very least, it does duplicate functionality, and this could be made much more concise.

An 8 byte change `|| !key1` would have the same impact, and absolutely avoid breaking logic