apache / apache/pulsar

Improve testing for authorization to prevent regressions in topic operations related to subscriptions

Open
#19,183 4 comments 0 reactions 1 assignee Claimed by @michaeljmarshall View on GitHub
area/authz Stale type/enhancement
Dominant language
Java
Stars
15.3k
Forks
3.8k
Avg merge
1d 14h
Merged PRs (30d)
160

Description

### Search before asking

- [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar.

### Motivation

In PR https://github.com/apache/pulsar/pull/19176, I fixed a minor regression in our authorization code base related to topic operations. We should have better tests or possibly a better design to make sure we catch these regressions before they are merged.

The tests should also cover cases seen in https://github.com/apache/pulsar/pull/17123 and in https://github.com/apache/pulsar/pull/19184

### Solution

Improve tests or improve the framework.

### Alternatives

_No response_

### Anything else?

_No response_

### Are you willing to submit a PR?

- [X] I'm willing to submit a PR!

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.