-
Notifications
You must be signed in to change notification settings - Fork 25.2k
[CI] SSLErrorMessageFileTests class failing #121960
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Pinging @elastic/es-security (Team:Security) |
Pinging @elastic/es-core-infra (Team:Core/Infra) |
This passes reliably without entitlements and fails with them enabled:
(Note that no seed is needed above). Based on the assertion failure, I'm assuming that we're missing entitlements-related config: Expected:
But was:
Could someone from Core Infra take a look? Thanks! |
This looks like a test-only problem. We need test config for entitlements to fix. |
@n1v0lg This is not entitlements related (note that although entitlements are in the jvm args, that is only used when running ES currently, not in unit tests such as this one). The problem is this test ran on JDK 24, where no security manager can be used. It looks like the test (or the ssl error message code) is specifically looking for security manager exceptions. It should be refactored to work without that. |
Good catch @rjernst -- on my plate to take another look 👀 |
@n1v0lg any progress on this? This blocks us from migrating to JDK 24 |
@mosche I'll take a look today or find someone else from the security team. |
Thanks @n1v0lg, I had a brief look. The failing tests rely on security manager denying the operation, this won't be possible using entitlements (at last at the moment). Entitlement checks are not enabled for unit tests, but only when running Elasticsearch (e.g. in REST tests). |
Fixing this issue will require a follow up by @elastic/es-security to migrate code from handling Respective tests would have to be implemented as REST tests |
SSL file utils currently only handle security manager access control exceptions around file read checks. This PR extends these to support entitlement checks as well. There is no easy way to unit test this since we can't run unit tests with entitlements enabled (for now). The PR includes a REST test instead. Relates: #121960
SSL file utils currently only handle security manager access control exceptions around file read checks. This PR extends these to support entitlement checks as well. There is no easy way to unit test this since we can't run unit tests with entitlements enabled (for now). The PR includes a REST test instead. Relates: elastic#121960
SSL file utils currently only handle security manager access control exceptions around file read checks. This PR extends these to support entitlement checks as well. There is no easy way to unit test this since we can't run unit tests with entitlements enabled (for now). The PR includes a REST test instead. Relates: elastic#121960
SSL file utils currently only handle security manager access control exceptions around file read checks. This PR extends these to support entitlement checks as well. There is no easy way to unit test this since we can't run unit tests with entitlements enabled (for now). The PR includes a REST test instead. Relates: #121960
SSL file utils currently only handle security manager access control exceptions around file read checks. This PR extends these to support entitlement checks as well. There is no easy way to unit test this since we can't run unit tests with entitlements enabled (for now). The PR includes a REST test instead. Relates: #121960
SSL file utils currently only handle security manager access control exceptions around file read checks. This PR extends these to support entitlement checks as well. There is no easy way to unit test this since we can't run unit tests with entitlements enabled (for now). The PR includes a REST test instead. Relates: #121960
SSL file utils currently only handle security manager access control exceptions around file read checks. This PR extends these to support entitlement checks as well. There is no easy way to unit test this since we can't run unit tests with entitlements enabled (for now). The PR includes a REST test instead. Relates: elastic#121960
Build Scans:
Reproduction Line:
Applicable branches:
main
Reproduces locally?:
N/A
Failure History:
See dashboard
Failure Message:
Issue Reasons:
Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.
The text was updated successfully, but these errors were encountered: