Skip to content

fix #1170: Use ReentrantLock instead of Object monitors #2264

fix #1170: Use ReentrantLock instead of Object monitors

fix #1170: Use ReentrantLock instead of Object monitors #2264

Triggered via pull request October 26, 2023 09:47
Status Failure
Total duration 27m 36s
Artifacts 15

ci.yml

on: pull_request
boringssl_clone
4s
boringssl_clone
Matrix: build
Matrix: openjdk-test
Fit to window
Zoom out
Zoom in

Annotations

81 errors and 19 warnings
build (windows-latest): testing/src/main/java/org/conscrypt/java/security/AbstractAlgorithmParametersTest.java#L28
warning: [PublicConstructorForAbstractClass] Constructors of on an abstract class can be declared protected as there is never a need for them to be public
build (windows-latest): testing/src/main/java/org/conscrypt/java/security/TestKeyStore.java#L701
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
build (windows-latest): testing/src/main/java/org/conscrypt/java/security/TestKeyStore.java#L702
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
build (windows-latest): testing/src/main/java/org/conscrypt/java/security/TestKeyStore.java#L943
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
build (windows-latest): testing/src/main/java/org/conscrypt/java/security/TestKeyStore.java#L965
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
build (windows-latest): testing/src/main/java/org/conscrypt/tlswire/handshake/HelloExtension.java#L95
warning: [EmptyBlockTag] A block tag (@param, @return, @throws, @deprecated) has an empty description. Block tags without descriptions don't add much value for future readers of the code; consider removing the tag entirely or adding a description.
build (windows-latest): testing/src/main/java/org/conscrypt/tlswire/handshake/HelloExtension.java#L95
warning: [MissingSummary] A summary line is required on public/protected Javadocs.
build (windows-latest): testing/src/main/java/org/conscrypt/VeryBasicHttpServer.java#L118
warning: [StringSplitter] String.split(String) has surprising behavior
build (windows-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L583
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (windows-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L584
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (windows-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L595
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (windows-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L596
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (windows-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L656
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (windows-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L659
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (windows-latest): common/src/test/java/org/conscrypt/java/security/cert/CertificateFactoryTest.java#L799
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
build (windows-latest): common/src/test/java/org/conscrypt/javax/net/ssl/HttpsURLConnectionTest.java#L193
warning: [CatchFail] Ignoring exceptions and calling fail() is unnecessary, and makes test output less useful
build (windows-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "58" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (windows-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "47" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (windows-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "47" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (windows-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "47" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (windows-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "47" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (windows-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "45" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (windows-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "47" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (windows-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "45" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (windows-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "45" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (windows-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "45" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (macos-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L583
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (macos-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L584
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (macos-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L595
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (macos-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L596
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (macos-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L656
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (macos-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L659
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (macos-latest): common/src/test/java/org/conscrypt/java/security/cert/CertificateFactoryTest.java#L799
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
build (macos-latest): common/src/test/java/org/conscrypt/javax/net/ssl/HttpsURLConnectionTest.java#L193
warning: [CatchFail] Ignoring exceptions and calling fail() is unnecessary, and makes test output less useful
build (macos-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "58" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (macos-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "47" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (macos-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "47" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (macos-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "47" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (macos-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "47" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (macos-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "45" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (macos-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "47" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (macos-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "45" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (macos-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "45" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (macos-latest): common/src/main/java/org/conscrypt/metrics/CipherSuite.java#L29
warning - Tag @see:illegal character: "45" in "https://www.iana.org/assignments/tls-parameters/tls-parameters.xhtml#tls-parameters-4"
build (macos-latest): testing/src/main/java/org/conscrypt/VeryBasicHttpServer.java#L118
warning: [StringSplitter] String.split(String) has surprising behavior
build (macos-latest): testing/src/main/java/org/conscrypt/java/security/TestKeyStore.java#L701
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
build (macos-latest): testing/src/main/java/org/conscrypt/java/security/TestKeyStore.java#L702
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
build (macos-latest): testing/src/main/java/org/conscrypt/java/security/TestKeyStore.java#L943
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
build (macos-latest): testing/src/main/java/org/conscrypt/java/security/TestKeyStore.java#L965
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
build (macos-latest): testing/src/main/java/org/conscrypt/tlswire/handshake/HelloExtension.java#L95
warning: [EmptyBlockTag] A block tag (@param, @return, @throws, @deprecated) has an empty description. Block tags without descriptions don't add much value for future readers of the code; consider removing the tag entirely or adding a description.
build (macos-latest): testing/src/main/java/org/conscrypt/tlswire/handshake/HelloExtension.java#L95
warning: [MissingSummary] A summary line is required on public/protected Javadocs.
build (macos-latest): testing/src/main/java/org/conscrypt/java/security/AbstractAlgorithmParametersTest.java#L28
warning: [PublicConstructorForAbstractClass] Constructors of on an abstract class can be declared protected as there is never a need for them to be public
build (ubuntu-latest): testing/src/main/java/org/conscrypt/java/security/TestKeyStore.java#L701
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
build (ubuntu-latest): testing/src/main/java/org/conscrypt/java/security/TestKeyStore.java#L702
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
build (ubuntu-latest): testing/src/main/java/org/conscrypt/java/security/TestKeyStore.java#L943
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
build (ubuntu-latest): testing/src/main/java/org/conscrypt/java/security/TestKeyStore.java#L965
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
build (ubuntu-latest): testing/src/main/java/org/conscrypt/tlswire/handshake/HelloExtension.java#L95
warning: [EmptyBlockTag] A block tag (@param, @return, @throws, @deprecated) has an empty description. Block tags without descriptions don't add much value for future readers of the code; consider removing the tag entirely or adding a description.
build (ubuntu-latest): testing/src/main/java/org/conscrypt/tlswire/handshake/HelloExtension.java#L95
warning: [MissingSummary] A summary line is required on public/protected Javadocs.
build (ubuntu-latest): testing/src/main/java/org/conscrypt/VeryBasicHttpServer.java#L118
warning: [StringSplitter] String.split(String) has surprising behavior
build (ubuntu-latest): testing/src/main/java/org/conscrypt/java/security/AbstractAlgorithmParametersTest.java#L28
warning: [PublicConstructorForAbstractClass] Constructors of on an abstract class can be declared protected as there is never a need for them to be public
build (ubuntu-latest): platform/src/main/java/org/conscrypt/TrustedCertificateStore.java#L117
warning: [UnnecessaryParentheses] These grouping parentheses are unnecessary; it is unlikely the code will be misinterpreted without them
build (ubuntu-latest): platform/src/main/java/org/conscrypt/TrustedCertificateStore.java#L355
warning: [InvalidBlockTag] Tag name `VisibleForTesting` is unknown. If this is a commonly-used custom tag, please click 'not useful' and file a bug.
build (ubuntu-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L583
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (ubuntu-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L584
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (ubuntu-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L595
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (ubuntu-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L596
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (ubuntu-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L656
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (ubuntu-latest): openjdk/src/test/java/org/conscrypt/ConscryptSocketTest.java#L659
warning: [deprecation] <T>assertThat(T,Matcher<? super T>) in Assert has been deprecated
build (ubuntu-latest): common/src/test/java/org/conscrypt/javax/net/ssl/HttpsURLConnectionTest.java#L193
warning: [CatchFail] Ignoring exceptions and calling fail() is unnecessary, and makes test output less useful
build (ubuntu-latest): common/src/test/java/org/conscrypt/java/security/cert/CertificateFactoryTest.java#L799
warning: [JdkObsolete] Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
openjdk-test (windows-latest, 8)
Process completed with exit code 1.
build (windows-latest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/[email protected], actions/checkout@v2, actions/setup-go@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
build (windows-latest)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
build (windows-latest)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
build (macos-latest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/[email protected], actions/checkout@v2, actions/setup-go@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
build (macos-latest)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
build (macos-latest)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
build (ubuntu-latest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/[email protected], actions/checkout@v2, actions/setup-go@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
build (ubuntu-latest)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
build (ubuntu-latest)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
uberjar
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
openjdk-test (ubuntu-latest, 8)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-java@v1, actions/download-artifact@v2, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
openjdk-test (ubuntu-latest, 9)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-java@v1, actions/download-artifact@v2, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
openjdk-test (macos-latest, 8)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-java@v1, actions/download-artifact@v2, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
openjdk-test (ubuntu-latest, 11)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-java@v1, actions/download-artifact@v2, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
openjdk-test (macos-latest, 11)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-java@v1, actions/download-artifact@v2, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
openjdk-test (macos-latest, 9)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-java@v1, actions/download-artifact@v2, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
openjdk-test (windows-latest, 11)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-java@v1, actions/download-artifact@v2, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
openjdk-test (windows-latest, 8)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-java@v1, actions/download-artifact@v2, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
openjdk-test (windows-latest, 9)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-java@v1, actions/download-artifact@v2, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Artifacts

Produced during runtime
Name Size
boringssl-source Expired
333 KB
m2repo-Linux Expired
4.15 MB
m2repo-Windows Expired
2.41 MB
m2repo-macOS Expired
3.94 MB
m2repo-uber Expired
13.8 MB
test-results-macos-latest-11 Expired
716 KB
test-results-macos-latest-8 Expired
716 KB
test-results-macos-latest-9 Expired
716 KB
test-results-ubuntu-latest-11 Expired
719 KB
test-results-ubuntu-latest-8 Expired
719 KB
test-results-ubuntu-latest-9 Expired
719 KB
test-results-windows-latest-11 Expired
744 KB
test-results-windows-latest-8 Expired
749 KB
test-results-windows-latest-9 Expired
744 KB
testjar Expired
21.2 MB