It is possible that the broadcasted message on a peer is received after Identity finishes processing the request - the same issue existed before this change, but was less likely to happen.

The idea of broadcasting the message before calling Identity was motivated exactly by this reasoning. We wanted peers to increase the chance of receiving the message while the peer was still on Identity to create a session if it doesn't exist potentially. However, as you noticed, they can still be too late because the RPC is processed relatively fast, so your change doesn't make it much worse in this matter.