SEZ9 commented on PR #11557: URL: https://github.com/apache/seatunnel/pull/11557#issuecomment-6621950956
Thanks for the follow-up push (`c5787154a16`, 11 commits on top of `41b5818d6b8`) and for the per-issue table and self re-review. One item from my earlier review that I could not locate in the portion of the table and re-review I can see is **PR11557-F1 (HIGH, Security) – unsafe Java deserialization pattern in `seatunnel-engine/seatunnel-engine-server/src/main/java/org/apache/seatunnel/engine/server/task/operation/source/ManagedSourceRegisterOperation.java`**. Since you mention one commit per issue across 11 commits, it may well be covered by one of the rows I can't see. Could you confirm whether F1 was addressed on `c5787154a16a8d00ec49da7e161bf3ea3dbe76a8`? If so, a pointer to the specific commit plus a short root-cause/fix/test note in the same format as the other rows would be great. If it is still open, please add it to the status table so the resolution state of every blocker is visible in one place. Once F1 is accounted for, I'll do a final pass on that single item. I'm not raising anything new here. <!-- streview-comment:2268 --> -- 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]
