Skip to content
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

Githubci and java17 #4534

Closed
wants to merge 48 commits into from
Closed

Conversation

siladu
Copy link
Contributor

@siladu siladu commented Oct 14, 2022

Combination of #4162 and #4532 (squashed and cherry-picked)

atoulme added 30 commits October 9, 2022 23:52
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
atoulme and others added 15 commits October 9, 2022 23:52
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Signed-off-by: Antoine Toulme <[email protected]>
Migrate the build to use Java 17 as the base version.

Signed-off-by: Danno Ferrin <[email protected]>

tweak docker files and tests

Signed-off-by: Danno Ferrin <[email protected]>

javadoc

turn off missing comment lint
fix header errors

Signed-off-by: Danno Ferrin <[email protected]>

update changed error

Signed-off-by: Danno Ferrin <[email protected]>

fixes

run codeql in Java 17
run jdk latest in a new debian version

Signed-off-by: Danno Ferrin <[email protected]>

tweak lgtm and arm64

Signed-off-by: Danno Ferrin <[email protected]>

undo cimg on arm64

Signed-off-by: Danno Ferrin <[email protected]>

spotless and codeql for access issues

update spotless to avoid opens and exports
update --add-opens for codeql check

Signed-off-by: Danno Ferrin <[email protected]>

add api opening to invocation

Signed-off-by: Danno Ferrin <[email protected]>

syntax

Signed-off-by: Danno Ferrin <[email protected]>

remove openjdk-latest docker image

Signed-off-by: Danno Ferrin <[email protected]>

missed some openjdk-latest

Signed-off-by: Danno Ferrin <[email protected]>

spotless

Signed-off-by: Justin Florentine <[email protected]>

using wrong image

Signed-off-by: Justin Florentine <[email protected]>
@SuppressWarnings("DoNotInvokeMessageDigestDirectly")
private static MessageDigest setUpDigest() {
try {
return MessageDigest.getInstance("SHA-1");

Check failure

Code scanning / CodeQL

Use of a potentially broken or risky cryptographic algorithm

Cryptographic algorithm [SHA-1](1) may not be secure, consider using a different algorithm.
@SuppressWarnings("DoNotInvokeMessageDigestDirectly")
private static MessageDigest setUpDigest() {
try {
return MessageDigest.getInstance("SHA-1");

Check failure

Code scanning / CodeQL

Use of a potentially broken or risky cryptographic algorithm

Cryptographic algorithm [SHA-1](1) may not be secure, consider using a different algorithm.
@siladu siladu closed this Nov 15, 2022
@siladu siladu deleted the githubci-and-java17 branch November 15, 2022 03:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants