Skip to content

Conversation

@scorgn
Copy link

@scorgn scorgn commented Oct 13, 2024

  • Fix seg fault in oauthbearerSetToken definition in rdkafka.c when not passing value for extension
  • Fix issue with oauthbearer_token_refresh not persisting (was not included in kafka_conf_callbacks_copy)
  • Modify start-kafka.sh to start kafka container with oauth2 configuration if librdkafka version supports it
  • Implement oauth2 integration tests, skipped for versions of librdkafka less than v1.1.0

* Fix seg fault in oauthbearerSetToken definition in rdkafka.c when not passing value for extension
* Fix issue with oauthbearer_token_refresh not persisting (was not included in kafka_conf_callbacks_copy)
* Modify start-kafka.sh to start kafka container with oauth2 configuration
* Implement oauth2 integration tests
@scorgn scorgn force-pushed the feat/oauthbearer-set-token branch from 4029bb4 to 4d3f493 Compare October 13, 2024 15:33
@scorgn scorgn changed the title Implement integration tests for oauth token methods Draft: Implement integration tests for oauth token methods Oct 13, 2024
… support it

* Add new test env var SKIP_OAUTH based on matrix.skipoauth
* Set matrix.skipoauth on all librdkafka versions below v1.1.0
* Don't set up kafka_oauth2 container if SKIP_OAUTH is 1
* Skip tests in oauthbearer_integration.phpt if RD_KAFKA_VERSION is below 0x010100
@scorgn scorgn changed the title Draft: Implement integration tests for oauth token methods Implement integration tests for oauth token methods Oct 14, 2024
@scorgn scorgn closed this Oct 15, 2024
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