Skip to content

Commit cba5eff

Browse files
authored
A couple more CSP enhancements (#1112)
1 parent c2db7be commit cba5eff

File tree

4 files changed

+5
-5
lines changed

4 files changed

+5
-5
lines changed

server/configs/application.properties

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ context.encryptionKey=@@encryptionKey@@
4646

4747
## By default, we serve LabKey at the root context path (e.g. http://localhost:8080)
4848
## You may customize the context path if you wish (e.g. http://localhost:8080/labkey)
49+
## The context path value must start with a slash
4950
#context.contextPath=@@contextPath@@
5051

5152
## Using a legacy context path provides backwards compatibility with old deployments. A typical use case would be to

server/embedded/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -161,7 +161,7 @@ project.publishing {
161161
artifact project.tasks.bootJar.outputs.files.singleFile
162162
pom {
163163
name = "LabKey Server Embedded"
164-
description = "LabKey classes for producing distributions with embedded TomCat."
164+
description = "Embedded Tomcat, Spring Boot, and the LabKey classes that configure these components"
165165
developers PomFileHelper.getLabKeyTeamDevelopers()
166166
licenses PomFileHelper.getApacheLicense()
167167
organization PomFileHelper.getLabKeyOrganization()

server/embedded/src/org/labkey/embedded/LabKeyServer.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ public static void main(String[] args)
4646
return;
4747
}
4848

49-
// Issue 40038: Ride-or-die Mode - default to shutting down by default in embedded deployment scenario
49+
// Issue 40038: Ride-or-die Mode - default to shutting down by default
5050
if (System.getProperty(TERMINATE_ON_STARTUP_FAILURE) == null)
5151
{
5252
System.setProperty(TERMINATE_ON_STARTUP_FAILURE, "true");
@@ -71,7 +71,7 @@ public static void main(String[] args)
7171
String baseCsp = """
7272
default-src 'self' ;
7373
connect-src 'self' ${CONNECTION.SOURCES} ;
74-
object-src 'none' ;
74+
object-src ${OBJECT.SOURCES} ; /* Substitution value defaults to 'none' unless overridden by an admin */
7575
style-src 'self' 'unsafe-inline' ${STYLE.SOURCES} ;
7676
img-src 'self' data: ${IMAGE.SOURCES} ;
7777
font-src 'self' data: ${FONT.SOURCES} ;
@@ -871,5 +871,4 @@ public void setKeyStore(String keyStore)
871871
this.keyStore = keyStore;
872872
}
873873
}
874-
875874
}

server/embedded/src/org/labkey/embedded/LabKeyTomcatServletWebServerFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -275,7 +275,7 @@ private void addContextProperty(StandardContext context, String value, String na
275275
}
276276
}
277277

278-
// Issue 48565: allow for JSON-formatted access logs in embedded tomcat
278+
// Issue 48565: allow for JSON-formatted access logs
279279
private void configureJsonAccessLogging(Tomcat tomcat, LabKeyServer.JsonAccessLog logConfig)
280280
{
281281
var v = new JsonAccessLogValve();

0 commit comments

Comments
 (0)