Skip to content

Conversation

@gaoxh
Copy link
Contributor

@gaoxh gaoxh commented Jan 8, 2026

Ⅰ. Describe what this PR did

allow configuring platform codes to prohibit MQ messages from accessing the database

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@gaoxh gaoxh changed the title feat: allow configuring platform codes to prohibit MQ messages from accessing the databaseApp role id extend feat: feat: allow configuring platform codes to prohibit MQ messages from accessing the database Jan 8, 2026
@gaoxh gaoxh changed the title feat: feat: allow configuring platform codes to prohibit MQ messages from accessing the database feat: allow configuring platform codes to prohibit MQ messages from accessing the database Jan 8, 2026
wodiwudi
wodiwudi previously approved these changes Jan 8, 2026
Copy link
Contributor

@wodiwudi wodiwudi left a comment

Choose a reason for hiding this comment

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

LGTM

@gaoxh gaoxh dismissed wodiwudi’s stale review January 8, 2026 06:10

The merge-base changed after approval.

@gaoxh gaoxh requested a review from wtt40122 January 8, 2026 06:15
Copy link
Contributor

@wtt40122 wtt40122 left a comment

Choose a reason for hiding this comment

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

+LGTM

Copy link
Contributor

@wodiwudi wodiwudi left a comment

Choose a reason for hiding this comment

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

LGTM

@wodiwudi wodiwudi merged commit b915d7f into master Jan 8, 2026
3 of 4 checks passed
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.

4 participants