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

Prepare Tomcat 10 update [do not merge] #740

Closed
wants to merge 2 commits into from
Closed

Conversation

strehle
Copy link
Member

@strehle strehle commented Jan 7, 2024

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/186794062

The labels on this github issue will be updated when the story is started.

@strehle strehle requested a review from a team January 7, 2024 11:27
Copy link
Contributor

@Tallicia Tallicia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reorganized items look good

@strehle strehle changed the title Prepare Tomcat 10 update Prepare Tomcat 10 update [do not merge] Jan 8, 2024
@strehle
Copy link
Member Author

strehle commented Jan 8, 2024

In my local docker tests with keystore similar config works with tomcat 9 and 10, so in 9 we dont need SSLConfig, if we go to tomcat10 then we need to solve this here - or this config works then with tomcat 10

@strehle strehle marked this pull request as draft January 16, 2024 09:42
# Conflicts:
#	jobs/uaa/templates/config/tomcat/tomcat.server.xml.erb
#	spec/compare/all-properties-tomcat-server.xml
@strehle
Copy link
Member Author

strehle commented Feb 18, 2025

dont see a reason for it anymore

@strehle strehle closed this Feb 18, 2025
@strehle strehle deleted the fix/issues/677 branch February 18, 2025 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

Add SSLHostConfig to server.xml - prepare for tomcat 10
3 participants