Skip to content

fix: configuration for async-migration consumer use security properties#1327

Merged
psmagin merged 2 commits intomasterfrom
async-migration-consumer-config
Mar 17, 2026
Merged

fix: configuration for async-migration consumer use security properties#1327
psmagin merged 2 commits intomasterfrom
async-migration-consumer-config

Conversation

@psmagin
Copy link
Copy Markdown
Collaborator

@psmagin psmagin commented Mar 16, 2026 •
edited
Loading

Purpose

Apply Kafka security config for async migration consumer

Changes Checklist

  • API Changes: Document any API paths, methods, request or response bodies changed, added, or removed.
  • Database Schema Changes: Indicate any database schema changes and their impact. Confirm that migration scripts were created.
  • Interface Version Changes: Indicate any changes to interface versions.
  • Interface Dependencies: Document added or removed dependencies.
  • Permissions: Document any changes to permissions.
  • Logging: Confirm that logging is appropriately handled.
  • Unit Testing: Confirm that changed classes were covered by unit tests.
  • Integration Testing: Confirm that changed logic was covered by integration tests.
  • Manual Testing: Confirm that changes were tested on local or dev environment.
  • NEWS: Confirm that the NEWS file is updated with relevant information about the changes made in this pull request.

Related Issues

https://folio-org.atlassian.net/browse/MODINVSTOR-1531

@psmagin psmagin force-pushed the async-migration-consumer-config branch from f8f08de to 5d33c9f Compare March 16, 2026 12:33
@psmagin psmagin marked this pull request as ready for review March 16, 2026 12:33
@psmagin psmagin requested a review from a team as a code owner March 16, 2026 12:33
@psmagin psmagin force-pushed the async-migration-consumer-config branch from 5d33c9f to 9fb4d98 Compare March 17, 2026 13:03
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Mar 17, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@psmagin psmagin merged commit b22e3ea into master Mar 17, 2026
18 checks passed
@psmagin psmagin deleted the async-migration-consumer-config branch March 17, 2026 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@viacheslavkol viacheslavkol viacheslavkol approved these changes

@SvitlanaKovalova1 SvitlanaKovalova1 SvitlanaKovalova1 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.

3 participants

@psmagin @viacheslavkol @SvitlanaKovalova1