antville/helma
antville/helma
1
1
Fork 0
Code Issues 5 Pull requests 7 Projects Releases 5 Activity Actions

Update Jetty packages (major) #103

Open
renovate[bot] wants to merge 46 commits from renovate/major-jetty-packages into main
pull from: renovate/major-jetty-packages
merge into: antville:main
Conversation 4 Commits 46 Files changed 15 +89 -64
renovate[bot] commented 2024-05-18 15:20:17 +00:00 (Migrated from github.com)
Copy link

Mend Renovate

This PR contains the following updates:

  • Migration of Servlet API from JavaX to Jakarta 5
  • Upgrade of Jetty Servlet API to version 12.0.9 (EE9)
  • Upgrade of Apache File Upload API
Package Change Age Adoption Passing Confidence
org.eclipse.jetty:jetty-xml (source) 9.4.54.v20240208 -> 12.0.9 age adoption passing confidence
org.eclipse.jetty:jetty-servlet (source) 9.4.54.v20240208 -> 11.0.21 age adoption passing confidence

Warning

Some dependencies could not be looked up. Check the Dependency Dashboard for more information.

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: * Migration of Servlet API from JavaX to Jakarta 5 * Upgrade of Jetty Servlet API to version 12.0.9 (EE9) * Upgrade of Apache File Upload API | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [org.eclipse.jetty:jetty-xml](https://eclipse.dev/jetty) ([source](https://togithub.com/jetty/jetty.project)) | `9.4.54.v20240208` -> `12.0.9` | [![age](https://developer.mend.io/api/mc/badges/age/maven/org.eclipse.jetty:jetty-xml/12.0.9?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/maven/org.eclipse.jetty:jetty-xml/12.0.9?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/maven/org.eclipse.jetty:jetty-xml/9.4.54.v20240208/12.0.9?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/maven/org.eclipse.jetty:jetty-xml/9.4.54.v20240208/12.0.9?slim=true)](https://docs.renovatebot.com/merge-confidence/) | | [org.eclipse.jetty:jetty-servlet](https://eclipse.dev/jetty) ([source](https://togithub.com/jetty/jetty.project)) | `9.4.54.v20240208` -> `11.0.21` | [![age](https://developer.mend.io/api/mc/badges/age/maven/org.eclipse.jetty:jetty-servlet/11.0.21?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/maven/org.eclipse.jetty:jetty-servlet/11.0.21?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/maven/org.eclipse.jetty:jetty-servlet/9.4.54.v20240208/11.0.21?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/maven/org.eclipse.jetty:jetty-servlet/9.4.54.v20240208/11.0.21?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://togithub.com/renovatebot/renovate/discussions) if that's undesired. --- - [x] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/antville/helma). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zNjMuNSIsInVwZGF0ZWRJblZlciI6IjM3LjM2My41IiwidGFyZ2V0QnJhbmNoIjoiaGVsbWEt8J+QnCIsImxhYmVscyI6WyJjb3JlIiwiZGVwZW5kZW5jeSIsIm1ham9yIiwicnVudGltZSJdfQ==-->
renovate[bot] commented 2024-05-25 14:28:01 +00:00 (Migrated from github.com)
Copy link

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

### Edited/Blocked Notification Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. ⚠️ **Warning**: custom changes will be lost.
p3k commented 2024-05-30 22:02:46 +00:00 (Migrated from github.com)
Copy link

Looks like disabling and subsequently re-enabling an app in apps.properties does not restart the app…

Looks like disabling and subsequently re-enabling an app in `apps.properties` does not restart the app…
tobi added 1 commit 2025-01-10 18:58:55 +00:00 
# Conflicts:
#	build.gradle
tobi changed target branch from helma-🐜 to main 2025-01-10 19:01:18 +00:00
tobi force-pushed renovate/major-jetty-packages from 78d07cecce to 1925916220 2025-01-10 19:08:23 +00:00 Compare
tobi added 1 commit 2025-01-10 19:10:31 +00:00 
# Conflicts:
#	build.gradle
tobi added 1 commit 2025-02-15 20:07:47 +00:00
tobi commented 2025-04-07 15:18:03 +00:00
Owner
Copy link

@tobi wrote in #103 (comment):

Looks like disabling and subsequently re-enabling an app in apps.properties does not restart the app…

Fixed in 2994a4becc.

@tobi wrote in https://code.host.antville.org/antville/helma/pulls/103#issuecomment-1825: > Looks like disabling and subsequently re-enabling an app in `apps.properties` does not restart the app… Fixed in 2994a4becc41c97707bda88e08f64e3f8379da2e.
tobi added 15 commits 2025-04-10 20:52:48 +00:00 
This remediates the exception “Shared scheduler not started” and restores the functionality of enabling an app in apps.properties – see #103 (comment)
Fingers crossed it won’t be missed
Resolve FIXME but without fixing the issue
All checks were successful
Build / build (push) Successful in 19s
Details
6723df912e 
I tried to make this work but it might need a more thorough rewrite that I just cannot do.

The worst that can happen is stopping and starting Helma apps adding redundant ServletContextHandlers to the ContextHandlerCollection until Helma is restarted.
tobi added 1 commit 2025-04-10 21:37:58 +00:00
Merge remote-tracking branch 'origin/main' into renovate/major-jetty-packages
All checks were successful
Build / build (push) Successful in 10s
Details
0cf9cf7af2
tobi added 1 commit 2025-04-10 21:42:21 +00:00
Make deploy script executable
All checks were successful
Build / build (push) Successful in 8s
Details
517735e53d
tobi added 1 commit 2025-04-10 21:59:46 +00:00
Enable UTF-8 in Java source files
All checks were successful
Build / build (push) Successful in 9s
Details
854df4e865
tobi force-pushed renovate/major-jetty-packages from 854df4e865 to 9dee4993f1 2025-04-10 22:06:10 +00:00 Compare
tobi added 1 commit 2025-04-11 09:45:27 +00:00
Fix regression of Java sources not being processed before compiling
All checks were successful
Build / build (push) Successful in 34s
Details
2d9b330cea
tobi force-pushed renovate/major-jetty-packages from 2d9b330cea to ee7d543811 2025-04-11 09:49:15 +00:00 Compare
tobi added 1 commit 2025-04-11 10:24:12 +00:00
Revert "Do not use non-ASCII charaters in Java source code, yet"
All checks were successful
Build / build (push) Successful in 13s
Details
2ec02c7940 
This reverts commit 9dee4993f1.
tobi force-pushed renovate/major-jetty-packages from 2ec02c7940 to 98854ac3c9 2025-04-11 10:25:39 +00:00 Compare
tobi added 1 commit 2025-04-11 10:36:02 +00:00
tobi force-pushed renovate/major-jetty-packages from 76189d2997 to 98854ac3c9 2025-04-11 11:17:01 +00:00 Compare
tobi force-pushed renovate/major-jetty-packages from 98854ac3c9 to 745ce5bfca 2025-04-11 11:23:12 +00:00 Compare
tobi commented 2025-04-11 11:45:14 +00:00
Owner
Copy link

Looks good at the test instance:

  • image and file upload as well as fetch from remote URL work
  • health page shows the Proxmox graphs
  • XML-RPC requests work
Looks good at the test instance: * image and file upload as well as fetch from remote URL work * health page shows the Proxmox graphs * XML-RPC requests work
tobi self-assigned this 2025-04-11 11:47:19 +00:00
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin renovate/major-jetty-packages:renovate/major-jetty-packages
git checkout renovate/major-jetty-packages

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout main
git merge --no-ff renovate/major-jetty-packages
git checkout renovate/major-jetty-packages
git rebase main
git checkout main
git merge --ff-only renovate/major-jetty-packages
git checkout renovate/major-jetty-packages
git rebase main
git checkout main
git merge --no-ff renovate/major-jetty-packages
git checkout main
git merge --squash renovate/major-jetty-packages
git checkout main
git merge --ff-only renovate/major-jetty-packages
git checkout main
git merge renovate/major-jetty-packages
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
blocker

   
core

   
dependency

Pull request that updates a dependency file

  
major

   
needs-work

   
runtime

   
security

   
urgent
No labels
blocker
core
dependency
major
needs-work
runtime
security
urgent
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
tobi
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: antville/helma#103
No description provided.
Delete branch "renovate/major-jetty-packages"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?