Skip to content

add support for oauth with schema registry#1645

Open
KyriosGN0 wants to merge 8 commits intokafbat:mainfrom
KyriosGN0:issue/1575
Open

add support for oauth with schema registry#1645
KyriosGN0 wants to merge 8 commits intokafbat:mainfrom
KyriosGN0:issue/1575

Conversation

@KyriosGN0
Copy link

@KyriosGN0 KyriosGN0 commented Jan 22, 2026

  • Breaking change? (if so, please describe the impact and migration path for existing application instances)

closes #1575

What changes did you make? (Give an overview)

Is there anything you'd like reviewers to focus on?

How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

  • No need to
  • Manually (please, describe, if necessary)
  • Unit checks
  • Integration checks
  • Covered by existing automation

Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
  • My changes generate no new warnings (e.g. Sonar is happy)
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged

Check out Contributing and Code of Conduct

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: AvivGuiser <avivguiser@gmail.com>
@kapybro kapybro bot added status/triage Issues pending maintainers triage status/triage/manual Manual triage in progress status/triage/completed Automatic triage completed and removed status/triage Issues pending maintainers triage labels Jan 22, 2026
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hi KyriosGN0! 👋

Welcome, and thank you for opening your first PR in the repo!

Please wait for triaging by our maintainers.

Please take a look at our contributing guide.

Signed-off-by: AvivGuiser <avivguiser@gmail.com>
@KyriosGN0 KyriosGN0 marked this pull request as ready for review January 22, 2026 16:14
@KyriosGN0 KyriosGN0 requested a review from a team as a code owner January 22, 2026 16:14
Signed-off-by: AvivGuiser <avivguiser@gmail.com>
Signed-off-by: AvivGuiser <avivguiser@gmail.com>
@KyriosGN0
Copy link
Author

the failed test is passing locally

Signed-off-by: AvivGuiser <avivguiser@gmail.com>
@KyriosGN0 KyriosGN0 requested a review from germanosin January 29, 2026 19:09
@KyriosGN0
Copy link
Author

hey @germanosin, about the docs PR, should i open it in parallel ?

String clientSecret;

Boolean tokenCacheEnabled = true;
Integer tokenRefreshBufferSeconds = 60; // Refresh token N seconds before expiry
Copy link
Member

Choose a reason for hiding this comment

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

let's use duration for it

* Stores tokens with expiration time and provides automatic expiration checking.
*/
@Slf4j
public class OAuthTokenCache {
Copy link
Member

Choose a reason for hiding this comment

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

Could we use just Caffeine for it?

Copy link
Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Author

Choose a reason for hiding this comment

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

@germanosin done!

…efresh buffer

Signed-off-by: AvivGuiser <avivguiser@gmail.com>
Signed-off-by: AvivGuiser <avivguiser@gmail.com>
@KyriosGN0 KyriosGN0 requested a review from germanosin February 6, 2026 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/triage/completed Automatic triage completed status/triage/manual Manual triage in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

support Schema registry auth via oauth2

2 participants