RFR: 8337723: Remove redundant tests from com/sun/security/sasl/gsskerb

Fernando Guallini fguallini at openjdk.org
Mon Aug 5 14:11:01 UTC 2024


There are 3 manual GSS-API/Kerberos tests that require a manual setup and were added to the problem list years ago:

- com/sun/security/sasl/gsskerb/**AuthOnly**-> Verifies that both client and server have completed the authentication process.
- com/sun/security/sasl/gsskerb/**NoSecurityLayer** -> Attempts to use wrap and unwrap to demonstrate that these operations should not be allowed without a negotiated security layer.
- com/sun/security/sasl/gsskerb/**ConfSecurityLayer** -> Demonstrates the use of the security layer by wrapping and unwrapping messages on both client and server sides.

These tests need manual setup of a KDC environment, service/user principals, keytabs, JAAS configuration, and script adjustments. All that is time consuming and error prone.

In contrast, the automated test sun/security/krb5/auto/**SaslBasic** already covers these scenarios and sets up the KDC with the necessary principals **automatically**, no manual setup is required. 
I've enhanced SaslBasic to include additional wrap/unwrap checks with and without a security layer. As a result, the 3 manual tests and their associated files are now redundand and can be removed, as they no longer provide additional value.

-------------

Commit messages:
 - refactoring
 - Merge branch 'master' into 8337723
 - remov gsskerb tests

Changes: https://git.openjdk.org/jdk/pull/20464/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=20464&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8337723
  Stats: 887 lines in 12 files changed: 56 ins; 817 del; 14 mod
  Patch: https://git.openjdk.org/jdk/pull/20464.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/20464/head:pull/20464

PR: https://git.openjdk.org/jdk/pull/20464



More information about the security-dev mailing list