-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Update Jetty lib-setuid for Jetty 12 #10279
Comments
A new Jetty lib-setuid 2.0.0-SNAPSHOT has been released, so a PR on this project can begin. |
If I have a chance to, I might take a stab and will submit a PR if I do. |
PR is incoming already, stand by. |
Opened PR #10280 |
@scantor Since you are bundling this, which environments do you support for setuid? We have historically provided a linux(x86) and osx library, we had sunos and some windows ones in the past as well. At this point we are just resurrecting the linux one for Jetty 12 but open to feedback on others. |
Mostly it's a "me" thing, on Linux, though we do mention it in our examples for using Jetty to run Shibboleth and I have always noted that I thought it was the simplest option overall. Most people these days do the proxying thing so they don't care much about it, either through a load balancer or Apache. I just have an allergy to layer 7 proxying and haven't needed to go that route. In an official capacity, we don't really support MacOS and definitely not any other Unix, and we only bundle on Windows, not Linux. We document Linux heavily and recommend Jetty there, but that's all to this point. Windows doesn't have the port restriction, though I wasn't aware there was really anything like this for Windows at all. We have continually struggled with procrun to service-ize the Jetty process and haven't found any better options, I'd be curious to know what you meant exactly by "some windows ones". |
Back in the days of Jetty 6 we had some builds for dos & windows & windows-95 |
Reminder: we need to consider jetty/jetty-toolchain#278 as an alternative. |
PR #10406 merged. |
I noted this is closed, but so far as I can tell, it's not bundled with Jetty 12. Is it dead after all, or published separately, or just available as source for those who want it? |
There are no longer libs for setuid, it's all JNA based now.. But if you install the
Example (with
|
@joakime is this documented? If not, please open an issue about it. |
Jetty version(s)
12.0.0
Java version/vendor
(use: java -version)
17
OS type/version
Linux
Description
The lib-setuid 1.0.4 is incompatible with Jetty 12.
We need to update lib-setuid to support Jetty 12.
The text was updated successfully, but these errors were encountered: