Update UrlFetch sample (#1249)

* update sample for java8 and to make steps clearer

* fix style and remove url fetch in pom

* style fixes

Author: averikitsch

appengine-java8/urlfetch/src/main/java/com/example/appengine/UrlFetchServlet.java

@@ -22,7 +22,6 @@
 import java.io.OutputStreamWriter;
 import java.net.HttpURLConnection;
 import java.net.URL;
-import java.net.URLEncoder;
 import javax.servlet.ServletException;
 import javax.servlet.annotation.WebServlet;
 import javax.servlet.http.HttpServlet;
@@ -35,8 +34,7 @@
 @WebServlet(
     name = "URLFetch",
     description = "URLFetch: Write low order IP address to Cloud SQL",
-    urlPatterns = "/urlfetch"
-)
+    urlPatterns = "/urlfetch")
 public class UrlFetchServlet extends HttpServlet {
 
   @Override
@@ -67,23 +65,33 @@ public void doPost(HttpServletRequest req, HttpServletResponse resp)
     String id = req.getParameter("id");
     String text = req.getParameter("text");
 
-    if (id == null || text == null || id == "" || text == "") {
+    // Validation for id and text inputs.
+    if (id == null
+        || text == null
+        || id.isEmpty()
+        || text.isEmpty()
+        || Integer.parseInt(id) > 100) {
       req.setAttribute("error", "invalid input");
       req.getRequestDispatcher("/urlfetchresult.jsp").forward(req, resp);
       return;
     }
 
-    JSONObject jsonObj =
-        new JSONObject().put("userId", 1).put("id", id).put("title", text).put("body", text);
-
     // [START complex]
     URL url = new URL("http://jsonplaceholder.typicode.com/posts/" + id);
     HttpURLConnection conn = (HttpURLConnection) url.openConnection();
+    // Enable output for the connection.
     conn.setDoOutput(true);
+    conn.setRequestProperty("Content-Type", "application/json; charset=UTF-8");
+    conn.setRequestProperty("Accept", "application/json");
+    // Set HTTP request method.
     conn.setRequestMethod("PUT");
 
+    // Create JSON request.
+    JSONObject jsonObj =
+        new JSONObject().put("userId", 1).put("id", id).put("title", text).put("body", text);
+
     OutputStreamWriter writer = new OutputStreamWriter(conn.getOutputStream());
-    writer.write(URLEncoder.encode(jsonObj.toString(), "UTF-8"));
+    writer.write(jsonObj.toString());
     writer.close();
 
     int respCode = conn.getResponseCode(); // New items get NOT_FOUND on PUT
@@ -92,6 +100,7 @@ public void doPost(HttpServletRequest req, HttpServletResponse resp)
       StringBuffer response = new StringBuffer();
       String line;
 
+      // Read input data stream.
       BufferedReader reader = new BufferedReader(new InputStreamReader(conn.getInputStream()));
       while ((line = reader.readLine()) != null) {
         response.append(line);

appengine-java8/urlfetch/src/main/webapp/urlfetchresult.jsp

@@ -26,23 +26,20 @@ limitations under the License.
     <c:if test="${not empty joke}">
         <h2>Joke: ${joke}</h2>
     </c:if>
-    <p /><p />
+
     <c:if test="${not empty error}">
-        <h2>${error}</h2>
+        <h2 style="color:red">${error}</h2>
+    </c:if>
+
+    <c:if test="${not empty response}">
+      <p>${response}</p>
     </c:if>
-    <p />
-    <c:if test="${not empty response}">${response}</c:if>
-    <p />
-    <p>
+
     <form method="post">
-        <label for="id">&nbsp;&nbsp;ID:</label><input type="text" name="id" value="777"/><br />
+        <label for="id">&nbsp;&nbsp;ID:</label><input type="text" name="id" value="1"/><br />
         <label for="text">Text:</label><input type="text" name="text" value="Lorem ipsum dolor sit amet, consectetur adipiscing elit."/><br />
         <input type="submit" value="Send"/>
     </form>
-    </p>
-    <c:if test="">
-
-    </c:if>
 </body>
 </html>
-<!-- [END base]-->
+<!-- [END base]-->