There's two reasons I've done it this way: firstly to match the pattern `buildIdentityString` uses, but I didn't realise that's because of legacy reasons, and secondly to allow mocking out the call to a static method in tests. Thoughts?