Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enhancement: Allow library user to provide some default GrpcChannelOptions. #92

Merged
merged 1 commit into from
Aug 4, 2023

Conversation

markdBC
Copy link
Contributor

@markdBC markdBC commented Aug 3, 2023

Description

Add WithDefaultGrpcChannelOptions(GrpcChannelOptions) method to allow library user to configure GrpcChannel instance used by CerbosClient.

Fixes ##91

@charithe
Copy link

charithe commented Aug 4, 2023

Thank you for the PR. Could you please sign-off your commit as follows:

git rebase HEAD~1 --signoff
git push --force-with-lease origin custom-grpc-channel-options

@markdBC markdBC force-pushed the custom-grpc-channel-options branch from 460f190 to db4b0be Compare August 4, 2023 08:58
@markdBC
Copy link
Contributor Author

markdBC commented Aug 4, 2023

@charithe Done!

Copy link
Member

@oguzhand95 oguzhand95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

src/Sdk/Cerbos/Sdk/Builder/CerbosClientBuilder.cs Outdated Show resolved Hide resolved
src/Sdk/Cerbos/Sdk/Builder/CerbosClientBuilder.cs Outdated Show resolved Hide resolved
src/Sdk/Cerbos/Sdk/Builder/CerbosClientBuilder.cs Outdated Show resolved Hide resolved
src/Sdk/Cerbos/Sdk/Builder/CerbosClientBuilder.cs Outdated Show resolved Hide resolved
src/Sdk/Cerbos/Sdk/Builder/CerbosClientBuilder.cs Outdated Show resolved Hide resolved
src/Sdk/Cerbos/Sdk/Builder/CerbosClientBuilder.cs Outdated Show resolved Hide resolved
@markdBC markdBC force-pushed the custom-grpc-channel-options branch from db4b0be to b6d98ce Compare August 4, 2023 10:05
@markdBC
Copy link
Contributor Author

markdBC commented Aug 4, 2023

@oguzhand95 No problem. I've made all your suggested changes.

@markdBC markdBC changed the title enhancement: Allow library user to provide default some default GrpcChannelOptions. enhancement: Allow library user to provide some default GrpcChannelOptions. Aug 4, 2023
@markdBC markdBC force-pushed the custom-grpc-channel-options branch from b6d98ce to 69b28c3 Compare August 4, 2023 10:09
@markdBC markdBC force-pushed the custom-grpc-channel-options branch from 69b28c3 to 6df1ebd Compare August 4, 2023 10:19
@oguzhand95 oguzhand95 self-requested a review August 4, 2023 10:21
@markdBC
Copy link
Contributor Author

markdBC commented Aug 4, 2023

Sorry about the repeated force pushes. Had some minor trouble with signature verification (not familiar with GitHub). All sorted.

@oguzhand95
Copy link
Member

Sorry about the repeated force pushes. Had some minor trouble with signature verification (not familiar with GitHub). All sorted.

No problem 🙂

@oguzhand95 oguzhand95 merged commit db895b1 into cerbos:main Aug 4, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants