JNSimba opened a new pull request, #70085:
URL: https://github.com/apache/doris/pull/70085

   ### What problem does this PR solve?
   
   Issue Number: N/A
   
   Related PR: #69617
   
   Problem Summary:
   
   Backport the final merged commit `c8a75b273e999370e354ed39ae65db2c483c0ad0` 
to `branch-4.1`. Require global ADMIN for HTTP/HTTPS JDBC driver URLs supplied 
through CDC job creation, alteration, and standalone `cdc_stream`. Preserve the 
caller's connection context while parsing job SQL, and carry over the 
associated unit and regression tests.
   
   Resolve the two unit-test conflicts by adapting the added assertions to this 
branch's JUnit 4 APIs. Direct calls replace JUnit 5's `assertDoesNotThrow`; 
unexpected exceptions still fail the tests. Production behavior matches the 
merged fix.
   
   ### Release note
   
   HTTP/HTTPS JDBC driver URLs in CDC jobs and `cdc_stream` require global 
ADMIN privilege. Local driver permissions remain unchanged.
   
   ### Check List (For Author)
   
   - Validation: `mvn -o checkstyle:check -pl fe-core` passed with zero 
violations; `git diff --check` passed.
   - Unit/regression tests and full compilation: Not run for this backport.
   - Behavior changed: Yes, remote CDC driver URLs require ADMIN.
   - Does this need documentation: Yes, the remote CDC driver permission 
requirement should be documented.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to