You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@age.apache.org by "jrgemignani (via GitHub)" <gi...@apache.org> on 2026/02/12 20:42:48 UTC

[PR] Fix security, CI, and dependency issues in JDBC driver [age]

jrgemignani opened a new pull request, #2331:
URL: https://github.com/apache/age/pull/2331

   Note: This PR was created with AI tools and a human.
   
   Security audit of the JDBC driver source code (16 main + 5 test Java files). The driver is a pure type-mapping layer (PGobject/agtype serialization only) with no query-building APIs, so it has minimal attack surface. No application-level vulnerabilities were found.
   
   Update all dependencies to address critical vulnerabilities:
   
   - postgresql 42.6.0 -> 42.7.5: fixes CVE-2024-1597 (Critical, CVSS 10.0: SQL injection via line comment generation in simple query mode)
   - commons-text 1.10.0 -> 1.13.0: maintenance update (1.10.0 was not affected by CVE-2022-42889)
   - antlr4 4.12.0 -> 4.13.2: bug fixes, Java 17+ improvements
   - junit-jupiter 5.9.3 -> 5.11.4: maintenance update
   - testcontainers 1.18.0 -> 1.20.4: stability improvements
   - slf4j 2.0.7 -> 2.0.16: maintenance update
   - Gradle wrapper 7.0 -> 8.5: Java 17/21 build support
   
   Note: postgresql 42.7.7+ also fixes CVE-2025-49146 (High: auth bypass with channelBinding=require). Version 42.7.5 was chosen as a conservative update to fix the Critical CVE while minimizing risk from protocol changes in newer releases.
   
   Fix root cause of CI test flakiness in BaseDockerizedTest:
   - Add testcontainers wait strategy (Wait.forLogMessage) to ensure PostgreSQL is fully ready before attempting JDBC connection
   - Add sslmode=disable to JDBC URL, required because postgresql 42.7.x attempts SSL negotiation by default and the AGE Docker image does not have SSL configured
   - Remove silent exception swallowing (catch/println) on connection failure so test setup errors are reported immediately
   
   modified:   drivers/jdbc/gradle/wrapper/gradle-wrapper.properties
   modified:   drivers/jdbc/lib/build.gradle.kts
   modified:   drivers/jdbc/lib/src/test/java/org/apache/age/jdbc/BaseDockerizedTest.java


-- 
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: commits-unsubscribe@age.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix security, CI, and dependency issues in JDBC driver [age]

Posted by "jrgemignani (via GitHub)" <gi...@apache.org>.
jrgemignani closed pull request #2331: Fix security, CI, and dependency issues in JDBC driver
URL: https://github.com/apache/age/pull/2331


-- 
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: commits-unsubscribe@age.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix security, CI, and dependency issues in JDBC driver [age]

Posted by "jrgemignani (via GitHub)" <gi...@apache.org>.
jrgemignani commented on PR #2331:
URL: https://github.com/apache/age/pull/2331#issuecomment-3894260354

   This will have to be reworked.


-- 
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: commits-unsubscribe@age.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org