This fixes a bunch of cases where we did not need to return the error early at all and potentially roll back the state.
For example, the TripleManager::poke would return the first error, but that did not need to happen as erroring out on a triple generator should just bin the generator and not interfere with the poking of other generators.
Also adds in a check for triple_id or presignature_id collisions.
Show Destroy Plan
```
No changes. No objects need to be destroyed.
Either you have not created any objects yet or the existing objects were
already deleted outside of Terraform.
Destroy complete! Resources: 0 destroyed.
```
This fixes a bunch of cases where we did not need to return the error early at all and potentially roll back the state.
For example, the
TripleManager::poke
would return the first error, but that did not need to happen as erroring out on a triple generator should just bin the generator and not interfere with the poking of other generators.Also adds in a check for triple_id or presignature_id collisions.