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.

Sign up for GitHub

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

Jump to bottom

Ensure algorithm query param is passed for CSFLE #1373

Merged
merged 2 commits into from
Jan 3, 2025
Merged

Ensure algorithm query param is passed for CSFLE #1373

merged 2 commits into from
Jan 3, 2025

Conversation

rayokota
Copy link
Member

@rayokota rayokota commented Jan 3, 2025 •
edited
Loading

What

Ensure algorithm query param is passed for CSFLE

Checklist

  • Contains customer facing changes? Including API/behavior changes
  • Did you add sufficient unit test and/or integration test coverage for this PR?
    • If not, please explain why it is not required

References

JIRA:

Test & Review

Open questions / Follow-ups

@rayokota rayokota requested review from a team as code owners January 3, 2025 17:42
@sonarqube-confluent

This comment has been minimized.

Sign in to view
1 similar comment
@sonarqube-confluent
Copy link

Failed

  • 0.00% Coverage on New Code (is less than 80.00%)

Analysis Details

13 Issues

  • Bug 0 Bugs
  • Vulnerability 0 Vulnerabilities
  • Code Smell 13 Code Smells

Coverage and Duplications

  • Coverage 0.00% Coverage (57.40% Estimated after merge)
  • Duplications No duplication information (0.00% Estimated after merge)

Project ID: confluent-kafka-go

View in SonarQube

@rayokota rayokota merged commit 1f31b8e into master Jan 3, 2025
1 of 2 checks passed
@rayokota rayokota deleted the algo-param branch January 3, 2025 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Claimundefine Claimundefine Claimundefine approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rayokota @Claimundefine