Skip to content

Use Java 21 to build and run proxy, add a main entrypoint and a docker image#82

Open
f-necas wants to merge 6 commits into
geosolutions-it:masterfrom
georchestra:update-and-add-docker-image
Open

Use Java 21 to build and run proxy, add a main entrypoint and a docker image#82
f-necas wants to merge 6 commits into
geosolutions-it:masterfrom
georchestra:update-and-add-docker-image

Conversation

@f-necas
Copy link
Copy Markdown
Contributor

@f-necas f-necas commented May 18, 2026

This PR:

  • Bumps Java from 8 to 21 and use Jakarta instead of javax
  • Add a main entrypoint
  • Create a docker image with to allow a standalone http-proxy

CI is failing cause of java version use in repo's workflow. We could also set java 17 for minimal version

@tdipisa tdipisa requested review from axl8713 and removed request for offtherailz May 25, 2026 11:21
@tdipisa tdipisa assigned axl8713 and unassigned offtherailz May 25, 2026
@axl8713
Copy link
Copy Markdown
Collaborator

axl8713 commented May 29, 2026

@f-necas could you please also set JDK 21 in the CI scripts?

@f-necas f-necas force-pushed the update-and-add-docker-image branch from bfdefcf to f1c018e Compare May 29, 2026 12:52
@f-necas
Copy link
Copy Markdown
Contributor Author

f-necas commented May 29, 2026

@f-necas could you please also set JDK 21 in the CI scripts?

@axl8713 Done :)

@axl8713
Copy link
Copy Markdown
Collaborator

axl8713 commented Jun 1, 2026

@f-necas
Thanks for the PR and for taking care of the CI updates.

We've decided to keep Java 17 as the minimum supported version for now, so could you please update the PR accordingly and switch the baseline from Java 21 back to Java 17?

Sorry for the change in direction.

Thanks again for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants