Fix batch writes

Other corrections

Minor corrections

Address feedback

My concern gets addressed in a later diff

Address feedback

Address feedback

Rebase on master

OutboundKeyserverResponse can be deleted from identity_client.proto, right?

Remove OutboundKeyserverResponse from identity_client.proto

Been a couple years, but this looks mostly like what I remember

cmake looks fine to me

Address feedback

Address feedback

Address feedback

Apply feedback

Address feedback

Correct rebase

Still need to apply the original feedback, just updating the work to use the identity credentials

Rebase on keyserver+identity work

apply camelCase, fix returning a promise vs return a closure (void).

Rebase on master

Rebase on master

I rebased this into D8476. But I applied your suggestion @kamil, thanks

Rebase on master

Rebase on master

Rebase on master

Address feedback

Apply missed feedback during refactor

i don't love that we block on verify_access_token in the interceptor function. would rather use a tower layer instead of a tonic interceptor to call the code asynchronously,

Eventually, we should move this to the authenticated service

Makes sense as a transitional step. But I wonder if there is a better solution which someone else might know.

My cpp is very rusty, but shouldn't we have a header file? how will we call methods defined here?

Makes sense to me

squashed this into D7691 because of rebase work

squashed this into D7691 because of rebase work

LGTM if we just want node>=18 (as mentioned in summary)

Squash initial session commits into one