make sessionbean cafe as stateless

Azure-Samples · Sep 30, 2020 · a1b73b7 · a1b73b7
1 parent e96999b
commit a1b73b7
Show file tree

Hide file tree

Showing 15 changed files with 84 additions and 46 deletions.
diff --git a/1-start/src/main/java/cafe/web/view/Cafe.java b/1-start/src/main/java/cafe/web/view/Cafe.java
@@ -1,5 +1,6 @@
 package cafe.web.view;
 
+import java.io.IOException;
 import java.io.Serializable;
 import java.lang.invoke.MethodHandles;
 import java.net.InetAddress;
@@ -60,9 +61,14 @@ public void setPrice(Double price) {
 	}
 
 	public List<Coffee> getCoffeeList() {
+		this.getAllCoffees();
 		return coffeeList;
 	}
 
+    public String getHostName() {
+        return "true".equals(System.getenv("SHOW_HOST_NAME")) ? System.getenv("HOSTNAME") : "";
+    }
+
 	@PostConstruct
 	private void init() {
 		try {
@@ -79,7 +85,6 @@ public boolean verify(String hostname, SSLSession session) {
 					return true;
 				}
 			}).build();
-			this.getAllCoffees();
 		} catch (IllegalArgumentException | NullPointerException | WebApplicationException | UnknownHostException ex) {
 			logger.severe("Processing of HTTP response failed.");
 			ex.printStackTrace();
@@ -92,16 +97,16 @@ private void getAllCoffees() {
 				});
 	}
 
-	public void addCoffee() {
+	public void addCoffee() throws IOException {
 		Coffee coffee = new Coffee(this.name, this.price);
 		this.client.target(baseUri).request(MediaType.APPLICATION_JSON).post(Entity.json(coffee));
 		this.name = null;
 		this.price = null;
-		this.getAllCoffees();
+		FacesContext.getCurrentInstance().getExternalContext().redirect("/");
 	}
 
-	public void removeCoffee(String coffeeId) {
+	public void removeCoffee(String coffeeId) throws IOException {
 		this.client.target(baseUri).path(coffeeId).request().delete();
-		this.getAllCoffees();
+		FacesContext.getCurrentInstance().getExternalContext().redirect("/");
 	}
 }
diff --git a/1-start/src/main/webapp/index.xhtml b/1-start/src/main/webapp/index.xhtml
@@ -13,6 +13,9 @@
 		crossorigin="anonymous" />
 </h:head>
 <h:body>
+    <div>
+        <h:outputText value="#{cafe.hostName}" />
+    </div>
 	<h:form>
 		<div class="container">
 			<h2>

diff --git a/2-simple/src/main/java/cafe/web/view/Cafe.java b/2-simple/src/main/java/cafe/web/view/Cafe.java
@@ -1,5 +1,6 @@
 package cafe.web.view;
 
+import java.io.IOException;
 import java.io.Serializable;
 import java.lang.invoke.MethodHandles;
 import java.net.InetAddress;
@@ -60,9 +61,14 @@ public void setPrice(Double price) {
 	}
 
 	public List<Coffee> getCoffeeList() {
+		this.getAllCoffees();
 		return coffeeList;
 	}
 
+    public String getHostName() {
+        return "true".equals(System.getenv("SHOW_HOST_NAME")) ? System.getenv("HOSTNAME") : "";
+    }
+
 	@PostConstruct
 	private void init() {
 		try {
@@ -79,7 +85,6 @@ public boolean verify(String hostname, SSLSession session) {
 					return true;
 				}
 			}).build();
-			this.getAllCoffees();
 		} catch (IllegalArgumentException | NullPointerException | WebApplicationException | UnknownHostException ex) {
 			logger.severe("Processing of HTTP response failed.");
 			ex.printStackTrace();
@@ -92,16 +97,16 @@ private void getAllCoffees() {
 				});
 	}
 
-	public void addCoffee() {
+	public void addCoffee() throws IOException {
 		Coffee coffee = new Coffee(this.name, this.price);
 		this.client.target(baseUri).request(MediaType.APPLICATION_JSON).post(Entity.json(coffee));
 		this.name = null;
 		this.price = null;
-		this.getAllCoffees();
+		FacesContext.getCurrentInstance().getExternalContext().redirect("/");
 	}
 
-	public void removeCoffee(String coffeeId) {
+	public void removeCoffee(String coffeeId) throws IOException {
 		this.client.target(baseUri).path(coffeeId).request().delete();
-		this.getAllCoffees();
+		FacesContext.getCurrentInstance().getExternalContext().redirect("/");
 	}
 }
diff --git a/2-simple/src/main/webapp/index.xhtml b/2-simple/src/main/webapp/index.xhtml
@@ -13,6 +13,9 @@
 		crossorigin="anonymous" />
 </h:head>
 <h:body>
+    <div>
+        <h:outputText value="#{cafe.hostName}" />
+    </div>
 	<h:form>
 		<div class="container">
 			<h2>

diff --git a/3-integration/aad-ldap/src/main/java/cafe/web/view/Cafe.java b/3-integration/aad-ldap/src/main/java/cafe/web/view/Cafe.java
@@ -1,5 +1,6 @@
 package cafe.web.view;
 
+import java.io.IOException;
 import java.io.Serializable;
 import java.lang.invoke.MethodHandles;
 import java.net.InetAddress;
@@ -68,6 +69,7 @@ public void setPrice(Double price) {
 	}
 
 	public List<Coffee> getCoffeeList() {
+		this.getAllCoffees();
 		return coffeeList;
 	}
 
@@ -79,6 +81,10 @@ public boolean isDisabledForDeletion() {
         return !securityContext.isCallerInRole("admin");
     }
 
+    public String getHostName() {
+        return "true".equals(System.getenv("SHOW_HOST_NAME")) ? System.getenv("HOSTNAME") : "";
+    }
+
 	@PostConstruct
 	private void init() {
 		try {
@@ -94,8 +100,7 @@ private void init() {
 				public boolean verify(String hostname, SSLSession session) {
 					return true;
 				}
-            }).build().register(filter);
-			this.getAllCoffees();
+			}).build().register(filter);
 		} catch (IllegalArgumentException | NullPointerException | WebApplicationException | UnknownHostException ex) {
 			logger.severe("Processing of HTTP response failed.");
 			ex.printStackTrace();
@@ -108,16 +113,16 @@ private void getAllCoffees() {
 				});
 	}
 
-	public void addCoffee() {
+	public void addCoffee() throws IOException {
 		Coffee coffee = new Coffee(this.name, this.price);
 		this.client.target(baseUri).request(MediaType.APPLICATION_JSON).post(Entity.json(coffee));
 		this.name = null;
 		this.price = null;
-		this.getAllCoffees();
+		FacesContext.getCurrentInstance().getExternalContext().redirect("/");
 	}
 
-	public void removeCoffee(String coffeeId) {
+	public void removeCoffee(String coffeeId) throws IOException {
 		this.client.target(baseUri).path(coffeeId).request().delete();
-		this.getAllCoffees();
+		FacesContext.getCurrentInstance().getExternalContext().redirect("/");
 	}
 }
diff --git a/3-integration/aad-ldap/src/main/webapp/index.xhtml b/3-integration/aad-ldap/src/main/webapp/index.xhtml
@@ -13,6 +13,9 @@
 		crossorigin="anonymous" />
 </h:head>
 <h:body>
+    <div>
+        <h:outputText value="#{cafe.hostName}" />
+    </div>
 	<h:form>
 		<div class="container">
 			<h2>

diff --git a/3-integration/aad-oidc/src/main/java/cafe/web/view/Cafe.java b/3-integration/aad-oidc/src/main/java/cafe/web/view/Cafe.java
@@ -1,5 +1,6 @@
 package cafe.web.view;
 
+import java.io.IOException;
 import java.io.Serializable;
 import java.lang.invoke.MethodHandles;
 import java.net.InetAddress;
@@ -71,6 +72,7 @@ public void setPrice(Double price) {
 	}
 
 	public List<Coffee> getCoffeeList() {
+		this.getAllCoffees();
 		return coffeeList;
 	}
 
@@ -82,6 +84,10 @@ public boolean isDisabledForDeletion() {
         return !jwtUtil.isUserInAdminGroup();
     }
 
+    public String getHostName() {
+        return "true".equals(System.getenv("SHOW_HOST_NAME")) ? System.getenv("HOSTNAME") : "";
+    }
+
 	@PostConstruct
 	private void init() {
 		try {
@@ -97,8 +103,7 @@ private void init() {
 				public boolean verify(String hostname, SSLSession session) {
 					return true;
 				}
-            }).build().register(filter);
-			this.getAllCoffees();
+			}).build().register(filter);
 		} catch (IllegalArgumentException | NullPointerException | WebApplicationException | UnknownHostException ex) {
 			logger.severe("Processing of HTTP response failed.");
 			ex.printStackTrace();
@@ -111,16 +116,16 @@ private void getAllCoffees() {
 				});
 	}
 
-	public void addCoffee() {
+	public void addCoffee() throws IOException {
 		Coffee coffee = new Coffee(this.name, this.price);
 		this.client.target(baseUri).request(MediaType.APPLICATION_JSON).post(Entity.json(coffee));
 		this.name = null;
 		this.price = null;
-		this.getAllCoffees();
+		FacesContext.getCurrentInstance().getExternalContext().redirect("/");
 	}
 
-	public void removeCoffee(String coffeeId) {
+	public void removeCoffee(String coffeeId) throws IOException {
 		this.client.target(baseUri).path(coffeeId).request().delete();
-		this.getAllCoffees();
+		FacesContext.getCurrentInstance().getExternalContext().redirect("/");
 	}
 }
diff --git a/3-integration/aad-oidc/src/main/webapp/index.xhtml b/3-integration/aad-oidc/src/main/webapp/index.xhtml
@@ -13,6 +13,9 @@
 		crossorigin="anonymous" />
 </h:head>
 <h:body>
+    <div>
+        <h:outputText value="#{cafe.hostName}" />
+    </div>
 	<h:form>
 		<div class="container">
 			<h2>

diff --git a/3-integration/connect-db/mssql/src/main/java/cafe/web/view/Cafe.java b/3-integration/connect-db/mssql/src/main/java/cafe/web/view/Cafe.java
@@ -1,5 +1,6 @@
 package cafe.web.view;
 
+import java.io.IOException;
 import java.io.Serializable;
 import java.lang.invoke.MethodHandles;
 import java.net.InetAddress;
@@ -60,14 +61,14 @@ public void setPrice(Double price) {
 	}
 
 	public List<Coffee> getCoffeeList() {
-	    this.getAllCoffees();
+		this.getAllCoffees();
 		return coffeeList;
 	}
 
-    public String getPodName() {
-        return "true".equals(System.getenv("SHOW_POD_NAME")) ? System.getenv("HOSTNAME") : "";
+    public String getHostName() {
+        return "true".equals(System.getenv("SHOW_HOST_NAME")) ? System.getenv("HOSTNAME") : "";
     }
-	
+
 	@PostConstruct
 	private void init() {
 		try {
@@ -84,7 +85,6 @@ public boolean verify(String hostname, SSLSession session) {
 					return true;
 				}
 			}).build();
-			this.getAllCoffees();
 		} catch (IllegalArgumentException | NullPointerException | WebApplicationException | UnknownHostException ex) {
 			logger.severe("Processing of HTTP response failed.");
 			ex.printStackTrace();
@@ -97,14 +97,16 @@ private void getAllCoffees() {
 				});
 	}
 
-	public void addCoffee() {
+	public void addCoffee() throws IOException {
 		Coffee coffee = new Coffee(this.name, this.price);
 		this.client.target(baseUri).request(MediaType.APPLICATION_JSON).post(Entity.json(coffee));
 		this.name = null;
 		this.price = null;
+		FacesContext.getCurrentInstance().getExternalContext().redirect("/");
 	}
 
-	public void removeCoffee(String coffeeId) {
+	public void removeCoffee(String coffeeId) throws IOException {
 		this.client.target(baseUri).path(coffeeId).request().delete();
+		FacesContext.getCurrentInstance().getExternalContext().redirect("/");
 	}
 }
diff --git a/3-integration/connect-db/mssql/src/main/webapp/index.xhtml b/3-integration/connect-db/mssql/src/main/webapp/index.xhtml
@@ -14,7 +14,7 @@
 </h:head>
 <h:body>
     <div>
-        <h:outputText value="#{cafe.podName}" />
+        <h:outputText value="#{cafe.hostName}" />
     </div>
 	<h:form>
 		<div class="container">

diff --git a/3-integration/connect-db/openlibertyapplication-cluster.yaml b/3-integration/connect-db/openlibertyapplication-cluster.yaml
@@ -18,7 +18,7 @@ spec:
   # - replace "${DB_Type}" with "mssql" for testing DB connection with Azure SQL
   # - replace "${DB_Type}" with "postgres" for testing DB connection with Azure Database for PostgreSQL
   env:
-  - name: SHOW_POD_NAME
+  - name: SHOW_HOST_NAME
     value: 'true'
   - name: DB_SERVER_NAME
     valueFrom:

diff --git a/3-integration/connect-db/postgres/src/main/java/cafe/web/view/Cafe.java b/3-integration/connect-db/postgres/src/main/java/cafe/web/view/Cafe.java
@@ -1,5 +1,6 @@
 package cafe.web.view;
 
+import java.io.IOException;
 import java.io.Serializable;
 import java.lang.invoke.MethodHandles;
 import java.net.InetAddress;
@@ -60,14 +61,14 @@ public void setPrice(Double price) {
 	}
 
 	public List<Coffee> getCoffeeList() {
-	    this.getAllCoffees();
+		this.getAllCoffees();
 		return coffeeList;
 	}
 
-    public String getPodName() {
-        return "true".equals(System.getenv("SHOW_POD_NAME")) ? System.getenv("HOSTNAME") : "";
+    public String getHostName() {
+        return "true".equals(System.getenv("SHOW_HOST_NAME")) ? System.getenv("HOSTNAME") : "";
     }
-	
+
 	@PostConstruct
 	private void init() {
 		try {
@@ -84,7 +85,6 @@ public boolean verify(String hostname, SSLSession session) {
 					return true;
 				}
 			}).build();
-			this.getAllCoffees();
 		} catch (IllegalArgumentException | NullPointerException | WebApplicationException | UnknownHostException ex) {
 			logger.severe("Processing of HTTP response failed.");
 			ex.printStackTrace();
@@ -97,14 +97,16 @@ private void getAllCoffees() {
 				});
 	}
 
-	public void addCoffee() {
+	public void addCoffee() throws IOException {
 		Coffee coffee = new Coffee(this.name, this.price);
 		this.client.target(baseUri).request(MediaType.APPLICATION_JSON).post(Entity.json(coffee));
 		this.name = null;
 		this.price = null;
+		FacesContext.getCurrentInstance().getExternalContext().redirect("/");
 	}
 
-	public void removeCoffee(String coffeeId) {
+	public void removeCoffee(String coffeeId) throws IOException {
 		this.client.target(baseUri).path(coffeeId).request().delete();
+		FacesContext.getCurrentInstance().getExternalContext().redirect("/");
 	}
 }
diff --git a/3-integration/connect-db/postgres/src/main/webapp/index.xhtml b/3-integration/connect-db/postgres/src/main/webapp/index.xhtml
@@ -14,7 +14,7 @@
 </h:head>
 <h:body>
     <div>
-        <h:outputText value="#{cafe.podName}" />
+        <h:outputText value="#{cafe.hostName}" />
     </div>
 	<h:form>
 		<div class="container">