confluent-kafka-python
confluent-kafka-python copied to clipboard
Added example with oauth/oidc example with CCloud logicalCluster and identityPoolId
The following example shows how to configure the producer with OAUTH. For Confluent Cloud where we need to pass extensions for "logicalCluster" and "identityPoolId" we need to pass some additional paramters.
This example is more specific to Confluent Cloud. Can we move it to a new file?
Just make sure that this example is tested.
the example was tested against confluent cloud
@pranavrth there seems to be two issues here:
- the original client example for oauth seems to be using the incorrect callbackhandler. it should be using the librd one - can we fix this asap?
- ccloud example is missing (for sasl extensions) - this is ok in a separate file, but we should create it
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.
:white_check_mark: pranavrth
:x: sarwarbhuiyan
You have signed the CLA already but the status is still pending? Let us recheck it.
@pranavrth there seems to be two issues here:
- the original client example for oauth seems to be using the incorrect callbackhandler. it should be using the librd one - can we fix this asap?
- ccloud example is missing (for sasl extensions) - this is ok in a separate file, but we should create it
@homestealth the ccloud example is here in my PR with two additional parameters that does the trick and since this is a cli example I've added optional parameters to the cli which make it work. It seems overkill to have a separate file for the exact same code to be copied.
I think it is fine to keep only one file but we should add a comment stating the two new configs are for CC case only.
Please also sign the CLA.
@emasab - Any comment from your side before we can merge this?
This is a valuable example, since it is config-based and removes the callback. How can we close this out?
Hi everyone, any plans to have this uploaded in some capacity? Ran into this issue today and this example would have been very helpful.
@sarwarbhuiyan - Can you please address my last comment to add some documentation? Please sign the CLA as well. We should merge this PR.
Hi team, two more customers have reported issues with our current example. GTS has been finding this pull and giving the customer the example here, any way we can move this to live?