-
Notifications
You must be signed in to change notification settings - Fork 136
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
Migrated and upgraded some libraries to jakarta namespace #285
base: master
Are you sure you want to change the base?
Conversation
@riperat - thank you for submitting a PR. |
@uhurusurfa As you know, Oracle owns the name "Java" trademark, and Java EE has been rebranded to Jakarta EE, necessitating a package relocation. So, since javax packages are deprecated by their Jakarta counterparts, migrating to the Jakarta namespace is a must to use current versions of Mail dependency. |
@riperat - thanks for the insight - I was not aware of the underlying issue of name-spacing. |
@riperat - the CI/CD pipeline is failing. This particular failure is related to Java cacerts that allow it to verify the downloaded packages for some maven zips and the only fix I have figured out is to upload a custm cacerts file that is the very latest available. Please overwrite the cacerts file in the root folder with a newer one and see if it fixes the failing tests. |
66331cf
to
556fa9f
Compare
Change the file .github/workflow/action.yml. For the Maven build run command, change it to this: Not sure why it is now needed but running the "clean" command fixes it on my test branch. |
Fix indentation
Debug CI/CD run
Further debug - Remove SSL debug and leave the exec out
More test debug
@uhurusurfa Is this change going to be introduced in the next release? I'm asking because I see that the CI is failing again after we fixed It. |
The change is incomplete and that is why there are failures. |
Hi, is there something I can do to make it ready for the next release, I'm asking because I need the change for a project I'm working on. |
The test HealthCheckTest.shouldRespondToHealthCheckOnNetModule() fails and needs to be fixed.