Use channel of pointers to objects for consistency

[illia-malachyn]

I had to change the access interface of some endpoints. I'm not sure if I should've done this

[illia-malachyn]

cc @peterargue

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 53.33333% with 35 lines in your changes missing coverage. Please review.

Project coverage is 57.47%. Comparing base (2f7fa8b) to head (68972d0).
Report is 28 commits behind head on master.

Files with missing lines	Patch %	Lines
access/grpc/client.go	0.00%	15 Missing ⚠️
access/grpc/convert/convert.go	28.57%	10 Missing ⚠️
access/mocks/Client.go	0.00%	8 Missing ⚠️
access/http/client.go	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #797      +/-   ##
==========================================
+ Coverage   56.59%   57.47%   +0.88%     
==========================================
  Files          35       35              
  Lines        7305     7454     +149     
==========================================
+ Hits         4134     4284     +150     
+ Misses       2670     2660      -10     
- Partials      501      510       +9     

Flag	Coverage Δ
unittests	57.47% <53.33%> (+0.88%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[peterargue]

added a comment about returning nil instead of an empty object. otherwise looks good

[janezpodhostnik]

Looks good, just some empty return types that could be nil instead.