It would probably be more preferable to have the ``storage.RecordDecoder`` implementation you're using convert the Protocol Buffer to a non-DTO type that has business logic.  This way, you can get rid of the ``ok`` check here and the pointer behaviors.  This is what I've done elsewhere.  Place it under ``model/``?