Skip to content

Commit ef05552

Browse files
timis1timis1
andauthored
JAVA-23118 Fix java-ee-8-security-api module (eugenp#14397)
* JAVA-23118 Fix java-ee-8-security-api module * JAVA-23118 Additional changes --------- Co-authored-by: timis1 <[email protected]>
1 parent df3f609 commit ef05552

File tree

6 files changed

+24
-25
lines changed

6 files changed

+24
-25
lines changed

security-modules/java-ee-8-security-api/app-auth-basic-store-db/src/main/java/com/baeldung/javaee/security/AdminServlet.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package com.baeldung.javaee.security;
22

3-
import javax.servlet.ServletException;
43
import javax.servlet.annotation.HttpConstraint;
54
import javax.servlet.annotation.ServletSecurity;
65
import javax.servlet.annotation.WebServlet;
@@ -14,7 +13,7 @@
1413
public class AdminServlet extends HttpServlet {
1514

1615
@Override
17-
protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
16+
protected void doGet(HttpServletRequest request, HttpServletResponse response) throws IOException {
1817
response.getWriter().append("User :" + request.getUserPrincipal().getName() + "\n");
1918
response.getWriter().append("User in Role user_role :" + request.isUserInRole("user_role") + "\n");
2019
response.getWriter().append("User in Role admin_role :" + request.isUserInRole("admin_role"));

security-modules/java-ee-8-security-api/app-auth-basic-store-db/src/main/java/com/baeldung/javaee/security/AppConfig.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22

33
import javax.enterprise.context.ApplicationScoped;
44
import javax.security.enterprise.authentication.mechanism.http.BasicAuthenticationMechanismDefinition;
5-
import javax.security.enterprise.authentication.mechanism.http.CustomFormAuthenticationMechanismDefinition;
65
import javax.security.enterprise.identitystore.DatabaseIdentityStoreDefinition;
76

87
@BasicAuthenticationMechanismDefinition(realmName = "defaultRealm")

security-modules/java-ee-8-security-api/app-auth-basic-store-db/src/main/java/com/baeldung/javaee/security/UserServlet.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,5 @@
11
package com.baeldung.javaee.security;
22

3-
import javax.annotation.security.DeclareRoles;
4-
import javax.inject.Inject;
5-
import javax.security.enterprise.SecurityContext;
63
import javax.servlet.ServletException;
74
import javax.servlet.annotation.HttpConstraint;
85
import javax.servlet.annotation.ServletSecurity;

security-modules/java-ee-8-security-api/app-auth-basic-store-db/src/main/liberty/config/server.xml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,10 @@
44
<feature>webProfile-8.0</feature>
55
</featureManager>
66

7+
<variable name="default.http.port" defaultValue="9080"/>
8+
<variable name="default.https.port" defaultValue="9443"/>
9+
<variable name="app.context.root" defaultValue="/"/>
10+
711
<httpEndpoint httpPort="${default.http.port}" httpsPort="${default.https.port}"
812
id="defaultHttpEndpoint" host="*"/>
913
</server>

security-modules/java-ee-8-security-api/pom.xml

Lines changed: 18 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -31,32 +31,31 @@
3131
</dependencies>
3232

3333
<build>
34+
<finalName>${project.artifactId}</finalName>
3435
<plugins>
3536
<plugin>
37+
<groupId>org.apache.maven.plugins</groupId>
3638
<artifactId>maven-war-plugin</artifactId>
39+
<version>3.3.2</version>
40+
</plugin>
41+
<plugin>
42+
<groupId>io.openliberty.tools</groupId>
43+
<artifactId>liberty-maven-plugin</artifactId>
44+
<version>3.8.2</version>
3745
<configuration>
38-
<failOnMissingWebXml>false</failOnMissingWebXml>
39-
<packagingExcludes>pom.xml</packagingExcludes>
46+
<serverName>guideServer</serverName>
4047
</configuration>
4148
</plugin>
4249
<plugin>
43-
<groupId>net.wasdev.wlp.maven.plugins</groupId>
44-
<artifactId>liberty-maven-plugin</artifactId>
45-
<version>${liberty-maven-plugin.version}</version>
50+
<groupId>org.apache.maven.plugins</groupId>
51+
<artifactId>maven-failsafe-plugin</artifactId>
52+
<version>3.1.2</version>
4653
<configuration>
47-
<install>
48-
<runtimeUrl>
49-
https://public.dhe.ibm.com/ibmdl/export/pub/software/openliberty/runtime/release/2018-09-05_2337/openliberty-18.0.0.3.zip
50-
</runtimeUrl>
51-
</install>
52-
<looseApplication>true</looseApplication>
53-
<installAppPackages>project</installAppPackages>
54-
<configFile>src/main/liberty/config/server.xml</configFile>
55-
<stripVersion>true</stripVersion>
56-
<bootstrapProperties>
54+
<systemPropertyVariables>
5755
<default.http.port>${defaultHttpPort}</default.http.port>
58-
<default.https.port>${defaultHttpsPort}</default.https.port>
59-
</bootstrapProperties>
56+
<http.port>${defaultHttpPort}</http.port>
57+
<war.name>${liberty.var.app.context.root}</war.name>
58+
</systemPropertyVariables>
6059
</configuration>
6160
</plugin>
6261
</plugins>
@@ -65,9 +64,10 @@
6564
<properties>
6665
<defaultHttpPort>9080</defaultHttpPort>
6766
<defaultHttpsPort>9443</defaultHttpsPort>
68-
<javaee-version>8.0</javaee-version>
67+
<javaee-version>8.0.1</javaee-version>
6968
<liberty-maven-plugin.version>2.3</liberty-maven-plugin.version>
7069
<openliberty-runtime.version>18.0.0.1</openliberty-runtime.version>
70+
<liberty.var.app.context.root>${project.artifactId}</liberty.var.app.context.root>
7171
</properties>
7272

7373
</project>

security-modules/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
<module>apache-shiro</module>
1818
<module>cas</module>
1919
<module>cloud-foundry-uaa</module>
20-
<!-- <module>java-ee-8-security-api</module> --> <!-- Fixing in JAVA-23118 -->
20+
<module>java-ee-8-security-api</module>
2121
<module>jee-7-security</module>
2222
<module>jjwt</module>
2323
<module>jwt</module>

0 commit comments

Comments
 (0)