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

[GWC-1344] Hide version info on embedded GWC home page #1345

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

sikeoka
Copy link
Contributor

@sikeoka sikeoka commented Jan 14, 2025

This PR hides version information and some other internal server information from the GeoServer embedded GeoWebCache home page unless the user is logged in as an administrator.

The version number is still visible in a stand-alone GWC

Issue: #1344

@@ -73,6 +73,14 @@ public void checkSecurity(TileLayer layer, @Nullable BoundingBox extent, @Nullab
}
}

/**
Copy link
Member

Choose a reason for hiding this comment

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

As far as I know, if GWC is deployed stand-alone, there are indeed no SecurityFilter, the only implementation I'm aware of is in GeoServer (which you have updated accordingly in the GS counterpart).

I'm asking because I don't want to give the wrong impression in the GWC release notes... the commit message should probably be something along the lines of "Support hiding GWC version when integrated in GeoServer".
(we use the commit messages to build the GWC release notes)

@jodygarnett jodygarnett merged commit 034162e into GeoWebCache:main Jan 23, 2025
11 checks passed
@jodygarnett
Copy link
Contributor

@aaime I have addressed your commit message feedback by "embedded GWC" to imply in GeoServer only.

@aaime aaime changed the title [GWC-1344] Hide version info on GWC home page [GWC-1344] Hide version info on embedded GWC home page Jan 23, 2025
@sikeoka sikeoka deleted the GWC-1344 branch January 23, 2025 17:33
jodygarnett pushed a commit to jodygarnett/geowebcache that referenced this pull request Jan 24, 2025
)

# Conflicts:
#	geowebcache/core/src/main/java/org/geowebcache/GeoWebCacheDispatcher.java
jodygarnett pushed a commit to jodygarnett/geowebcache that referenced this pull request Jan 24, 2025
)

# Conflicts:
#	geowebcache/core/src/main/java/org/geowebcache/GeoWebCacheDispatcher.java

# Conflicts:
#	geowebcache/core/src/test/java/org/geowebcache/GeoWebCacheDispatcherTest.java
jodygarnett pushed a commit that referenced this pull request Jan 24, 2025
# Conflicts:
#	geowebcache/core/src/main/java/org/geowebcache/GeoWebCacheDispatcher.java
jodygarnett pushed a commit that referenced this pull request Jan 24, 2025
# Conflicts:
#	geowebcache/core/src/main/java/org/geowebcache/GeoWebCacheDispatcher.java

# Conflicts:
#	geowebcache/core/src/test/java/org/geowebcache/GeoWebCacheDispatcherTest.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants