Skip to content

Adding Tomcat favicon to the Tomcat Manager webapp #876

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

Merged
merged 1 commit into from
Jul 28, 2025

Conversation

xsimo
Copy link
Contributor

@xsimo xsimo commented Jul 28, 2025

  1. Tomcat Manager is sometimes run on server on which there is no ROOT context, therefore no default favicon is not found. This commits adds the tomcat favicon to the manager webapp directory and references it from the HTML_HEADER_SECTION in Constants.java

1. Tomcat Manager is sometimes run on server on which there is no ROOT context, therefore no default favicon is not found. This commits adds the tomcat favicon to the manager webapp directory and references it from the HTML_HEADER_SECTION in Constants.java
@markt-asf markt-asf merged commit d6e79d4 into apache:9.0.x Jul 28, 2025
@markt-asf
Copy link
Contributor

Also forward-ported to 10.1.x, 11.0.x and 12.0.x

xsimo added a commit to Gtechna-Dev/tomcat that referenced this pull request Jul 31, 2025
improving previous merged apache#876 (commit d6e79d4) as per Konstantin Kolinko suggestions.
markt-asf pushed a commit that referenced this pull request Jul 31, 2025
improving previous merged #876 (commit d6e79d4) as per Konstantin Kolinko suggestions.
markt-asf pushed a commit that referenced this pull request Jul 31, 2025
improving previous merged #876 (commit d6e79d4) as per Konstantin Kolinko suggestions.
markt-asf pushed a commit that referenced this pull request Jul 31, 2025
improving previous merged #876 (commit d6e79d4) as per Konstantin Kolinko suggestions.
markt-asf pushed a commit that referenced this pull request Jul 31, 2025
improving previous merged #876 (commit d6e79d4) as per Konstantin Kolinko suggestions.
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.

2 participants