Skip to content

SASL_SSL OAUTHBEARER support for high level consumer #581

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

Open
wants to merge 4 commits into
base: 6.x
Choose a base branch
from

Conversation

Rastusik
Copy link

@Rastusik Rastusik commented Apr 15, 2025

This PR addresses #580 .

I also extracted the main logic into a separate oauthbearer.c file, so it's not copy pasted into 2 places.

I will be happy for any help or support or feedback, since this is the first time I'm working on a PHP extension.

EDIT: I also needed to implement the poll() method to KafkaConsumer because I had problems with calling getMetadata() and getCommittedOffsets().

@Rastusik
Copy link
Author

Rastusik commented Apr 16, 2025

hey @arnaud-lb Is there any way to make the pipeline run? Is there anything else that needs to be done to move forward with this PR? No pressure, I just am trying to figure out how things work here.

@oaattia
Copy link

oaattia commented Apr 19, 2025

Waiting for the one to be merged soon.

@Rastusik
Copy link
Author

Rastusik commented May 6, 2025

@oaattia Is there anything I can do to speed up the PR process?

@oaattia
Copy link

oaattia commented May 7, 2025

I don't know, I just need this feature :)
may be @arnaud-lb can help with that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants