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

scripts, maven pom.xml and class to help to load libraries from a jar #3

Open
wants to merge 118 commits into
base: trunk
Choose a base branch
from

Conversation

jfclere
Copy link
Contributor

@jfclere jfclere commented Sep 4, 2018

That requires a small add in tomcat-trunk load the class and call the method and loops loading the libraries list the method has returned.
If no one complains I will commit both later this week.

@markt-asf
Copy link
Contributor

The Java class would need to be added to trunk (and 8.5.x and 7.0.x?)
The other files look like they belong in native/build/jar or a similar location rather than the root of the repo.
What is the plan for ensuring these binaries are available on each of those platforms? How many committers have access to all four? (I have 3 and I could add a Solaris VM fairly easily.)

@jfclere
Copy link
Contributor Author

jfclere commented Sep 5, 2018 via email

@jfclere
Copy link
Contributor Author

jfclere commented Sep 5, 2018

The Java class would need to be added to trunk (and 8.5.x and 7.0.x?) the JarLoader needs to be in the same jar where the libraries are. Otherwise the logic can't find them and we don't want to list all the possible library name as the ldd or otool can give them at the build time.

@markt-asf
Copy link
Contributor

That might be problematic from a JPMS point of view that expects a package to appear in one and only one JAR.

@markt-asf
Copy link
Contributor

To check I understand this correctly, the plan is not for the Tomcat project to provide a single JAR for all platforms but to provide the hooks necessary for downstream packagers to package the native library (and dependencies) for their platform if they wish?

@jfclere
Copy link
Contributor Author

jfclere commented Sep 5, 2018 via email

@jfclere
Copy link
Contributor Author

jfclere commented Sep 5, 2018

well for windows we already have one binary, we can put it in the jar ;-)

@markt-asf markt-asf closed this Jun 18, 2019
@markt-asf
Copy link
Contributor

Re-opening after auto-close due to repo changes made during migration from svn to git.
If you are able to rebase this PR against master rather than trunk that would be helpful. If not, we#ll take care oif that when we review the PR.

@markt-asf markt-asf reopened this Jun 18, 2019
markt-asf and others added 20 commits June 18, 2019 20:11
Unable to complete build when build directory is outside of the source
tree.
Patch provided by Bob Huemmer.
Fix client certificate authentication when a certificate contains an AIA
extension without an OCSP URI.
Patch provided by Milind Takawale.
Correct reference to the X509_STORE being configured
Remove calls to X509_STORE_free since a separate store is not allocated
for CRLs. It simply re-uses the existing store for the server
SSLContext. Trying to close it triggers a JVM crash.
If the environment variable SSLKEYLOGFILE is set then the TLS keys will
be logged to that file.
Patch provided by John Kelly.
I don't think CVE-2019-1559 applies to Tomcat Native but err on the side
of caution.
markt-asf and others added 28 commits March 26, 2021 09:50
Don't apply tricks and rely on OS behavior to resolve addresses. It is the
caller's responsibility to properly process socket addresses if more than
one exists.

This mimics the behavior of HTTPd's listen.c for apr_sockaddr_info_get().
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.

8 participants