Modularize the image server

CHANGELOG.md

@@ -26,6 +26,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 - [2020-06-13] Modified AsyncAppender queue size to 1024 by [@lflare].
 - [2020-06-13] Bumped client version to 5 by [@lflare].
+- [2020-06-13] Modularized the image server [@carbotaniuman].
 
 ## [1.0.0-RC14] - 2020-06-12
 ### Fixed

src/main/java/mdnet/base/ServerHandler.java

@@ -42,7 +42,7 @@ public class ServerHandler {
 		params.put("secret", settings.getClientSecret());
 		params.put("port", settings.getClientPort());
 		params.put("disk_space", settings.getMaxCacheSizeMib() * 1024 * 1024 /* MiB to bytes */);
-		params.put("network_speed", settings.getMaxBurstRateKibPerSecond() * 1024 /*KiB to bytes */);
+		params.put("network_speed", settings.getMaxBurstRateKibPerSecond() * 1024 /* KiB to bytes */);
 		params.put("build_version", Constants.CLIENT_BUILD);
 
 		HttpResponse<ServerSettings> response = Unirest.post(SERVER_ADDRESS + "ping")

src/main/kotlin/mdnet/base/Netty.kt

@@ -56,7 +56,7 @@ class Netty(private val tls: ServerSettings.TlsCert, private val clientSettings:
 
         override fun start(): Http4kServer = apply {
             if (LOGGER.isInfoEnabled) {
-                LOGGER.info("Starting webserver with {} threads", clientSettings.threads)
+                LOGGER.info("Starting Netty with {} threads", clientSettings.threads)
             }
 
             val (mainCert, chainCert) = getX509Certs(tls.certificate)

src/main/kotlin/mdnet/base/web/Application.kt

@@ -1,254 +1,39 @@
 /* ktlint-disable no-wildcard-imports */
 package mdnet.base.web
 
-import mdnet.base.Constants
 import mdnet.base.Netty
 import mdnet.base.ServerSettings
 import mdnet.base.Statistics
 import mdnet.base.settings.ClientSettings
-import mdnet.cache.CachingInputStream
 import mdnet.cache.DiskLruCache
-import org.apache.http.client.config.CookieSpecs
-import org.apache.http.client.config.RequestConfig
-import org.apache.http.impl.client.HttpClients
-import org.http4k.client.ApacheClient
-import org.http4k.core.BodyMode
 import org.http4k.core.Method
-import org.http4k.core.Request
-import org.http4k.core.Response
-import org.http4k.core.Status
 import org.http4k.core.then
-import org.http4k.filter.CachingFilters
-import org.http4k.filter.MaxAgeTtl
 import org.http4k.filter.ServerFilters
-import org.http4k.lens.Path
 import org.http4k.routing.bind
 import org.http4k.routing.routes
 import org.http4k.server.Http4kServer
 import org.http4k.server.asServer
-import org.slf4j.LoggerFactory
-import java.io.BufferedInputStream
-import java.io.BufferedOutputStream
-import java.io.InputStream
-import java.security.MessageDigest
-import java.util.concurrent.Executors
 import java.util.concurrent.atomic.AtomicReference
-import javax.crypto.Cipher
-import javax.crypto.CipherInputStream
-import javax.crypto.CipherOutputStream
-import javax.crypto.spec.SecretKeySpec
-
-private val LOGGER = LoggerFactory.getLogger("Application")
-private const val THREADS_TO_ALLOCATE = 262144 // 2**18 // Honestly, no reason to not just let 'er rip. Inactive connections will expire on their own :D
 
 fun getServer(cache: DiskLruCache, serverSettings: ServerSettings, clientSettings: ClientSettings, statistics: AtomicReference<Statistics>): Http4kServer {
-    val executor = Executors.newCachedThreadPool()
+    val imageServer = ImageServer(cache, statistics, serverSettings.imageServer)
-
-    if (LOGGER.isInfoEnabled) {
-        LOGGER.info("Starting image retriever")
-    }
-
-    val client = ApacheClient(responseBodyMode = BodyMode.Stream, client = HttpClients.custom()
-            .setDefaultRequestConfig(RequestConfig.custom()
-                    .setCookieSpec(CookieSpecs.IGNORE_COOKIES)
-                    .setConnectTimeout(3000)
-                    .setSocketTimeout(3000)
-                    .setConnectionRequestTimeout(3000)
-                    .build())
-            .setMaxConnTotal(THREADS_TO_ALLOCATE)
-            .setMaxConnPerRoute(THREADS_TO_ALLOCATE)
-            .build())
-
-    val app = { dataSaver: Boolean ->
-        { request: Request ->
-            val chapterHash = Path.of("chapterHash")(request)
-            val fileName = Path.of("fileName")(request)
-
-            val sanitizedUri = if (dataSaver) {
-                "/data-saver"
-            } else {
-                "/data"
-            } + "/$chapterHash/$fileName"
-
-            if (LOGGER.isInfoEnabled) {
-                LOGGER.info("Request for $sanitizedUri received")
-            }
-
-            val rc4Bytes = if (dataSaver) {
-                md5Bytes("saver$chapterHash.$fileName")
-            } else {
-                md5Bytes("$chapterHash.$fileName")
-            }
-            val cacheId = printHexString(rc4Bytes)
-            statistics.getAndUpdate {
-                it.copy(requestsServed = it.requestsServed + 1)
-            }
-
-            // Netty doesn't do Content-Length or Content-Type, so we have the pleasure of doing that ourselves
-            fun respondWithImage(input: InputStream, length: String?, type: String, lastModified: String?, cached: Boolean): Response =
-                    Response(Status.OK)
-                            .header("Content-Type", type)
-                            .header("X-Content-Type-Options", "nosniff")
-                            .header(
-                                    "Cache-Control",
-                                    listOf("public", MaxAgeTtl(Constants.MAX_AGE_CACHE).toHeaderValue()).joinToString(", ")
-                            )
-                            .header("Timing-Allow-Origin", "https://mangadex.org")
-                            .let {
-                                if (length != null) {
-                                    it.body(input, length.toLong()).header("Content-Length", length)
-                                } else {
-                                    it.body(input).header("Transfer-Encoding", "chunked")
-                                }
-                            }
-                            .let {
-                                if (lastModified != null) {
-                                    it.header("Last-Modified", lastModified)
-                                } else {
-                                    it
-                                }
-                            }
-                            .let {
-                                if (cached) {
-                                    it.header("X-Cache", "HIT")
-                                } else {
-                                    it.header("X-Cache", "MISS")
-                                }
-                            }
-
-            val snapshot = cache.get(cacheId)
-            if (snapshot != null) {
-                // our files never change, so it's safe to use the browser cache
-                if (request.header("If-Modified-Since") != null) {
-                    statistics.getAndUpdate {
-                        it.copy(browserCached = it.browserCached + 1)
-                    }
-
-                    if (LOGGER.isInfoEnabled) {
-                        LOGGER.info("Request for $sanitizedUri cached by browser")
-                    }
-
-                    val lastModified = snapshot.getString(2)
-                    snapshot.close()
-
-                    Response(Status.NOT_MODIFIED)
-                            .header("Last-Modified", lastModified)
-                } else {
-                    statistics.getAndUpdate {
-                        it.copy(cacheHits = it.cacheHits + 1)
-                    }
-
-                    if (LOGGER.isInfoEnabled) {
-                        LOGGER.info("Request for $sanitizedUri hit cache")
-                    }
-
-                    respondWithImage(
-                            CipherInputStream(BufferedInputStream(snapshot.getInputStream(0)), getRc4(rc4Bytes)),
-                            snapshot.getLength(0).toString(), snapshot.getString(1), snapshot.getString(2),
-                        true
-                    )
-                }
-            } else {
-                statistics.getAndUpdate {
-                    it.copy(cacheMisses = it.cacheMisses + 1)
-                }
-
-                if (LOGGER.isInfoEnabled) {
-                    LOGGER.info("Request for $sanitizedUri missed cache")
-                }
-                val mdResponse = client(Request(Method.GET, "${serverSettings.imageServer}$sanitizedUri"))
-
-                if (mdResponse.status != Status.OK) {
-                    if (LOGGER.isTraceEnabled) {
-                        LOGGER.trace("Upstream query for $sanitizedUri errored with status {}", mdResponse.status)
-                    }
-                    mdResponse.close()
-                    Response(mdResponse.status)
-                } else {
-                    if (LOGGER.isTraceEnabled) {
-                        LOGGER.trace("Upstream query for $sanitizedUri succeeded")
-                    }
-
-                    val contentType = mdResponse.header("Content-Type")!!
-                    val contentLength = mdResponse.header("Content-Length")
-                    val lastModified = mdResponse.header("Last-Modified")
-
-                    val editor = cache.edit(cacheId)
-
-                    // A null editor means that this file is being written to
-                    // concurrently so we skip the cache process
-                    if (editor != null && contentLength != null && lastModified != null) {
-                        if (LOGGER.isTraceEnabled) {
-                            LOGGER.trace("Request for $sanitizedUri is being cached and served")
-                        }
-                        editor.setString(1, contentType)
-                        editor.setString(2, lastModified)
-
-                        val tee = CachingInputStream(
-                                mdResponse.body.stream,
-                                executor, CipherOutputStream(BufferedOutputStream(editor.newOutputStream(0)), getRc4(rc4Bytes))
-                        ) {
-                            // Note: if neither of the options get called/are in the log
-                            // check that tee gets closed and for exceptions in this lambda
-                            if (editor.getLength(0) == contentLength.toLong()) {
-                                if (LOGGER.isInfoEnabled) {
-                                    LOGGER.info("Cache download for $sanitizedUri committed")
-                                }
-
-                                editor.commit()
-                            } else {
-                                if (LOGGER.isInfoEnabled) {
-                                    LOGGER.info("Cache download for $sanitizedUri aborted")
-                                }
-
-                                editor.abort()
-                            }
-                        }
-                        respondWithImage(tee, contentLength, contentType, lastModified, false)
-                    } else {
-                        editor?.abort()
-
-                        if (LOGGER.isTraceEnabled) {
-                            LOGGER.trace("Request for $sanitizedUri is being served")
-                        }
-                        respondWithImage(mdResponse.body.stream, contentLength, contentType, lastModified, false)
-                    }
-                }
-            }
-        }
-    }
-
-    CachingFilters
 
     return catchAllHideDetails()
             .then(ServerFilters.CatchLensFailure)
             .then(addCommonHeaders())
             .then(
                 routes(
-                            "/data/{chapterHash}/{fileName}" bind Method.GET to app(false),
+                    "/data/{chapterHash}/{fileName}" bind Method.GET to imageServer.handler(dataSaver = false),
-                            "/data-saver/{chapterHash}/{fileName}" bind Method.GET to app(true),
+                    "/data-saver/{chapterHash}/{fileName}" bind Method.GET to imageServer.handler(dataSaver = true),
-                            "/{token}/data/{chapterHash}/{fileName}" bind Method.GET to app(false),
+                    "/{token}/data/{chapterHash}/{fileName}" bind Method.GET to imageServer.handler(
-                            "/{token}/data-saver/{chapterHash}/{fileName}" bind Method.GET to app(true)
+                        dataSaver = false,
+                        tokenized = true
+                    ),
+                    "/{token}/data-saver/{chapterHash}/{fileName}" bind Method.GET to imageServer.handler(
+                        dataSaver = false,
+                        tokenized = true
+                    )
                 )
             )
             .asServer(Netty(serverSettings.tls, clientSettings, statistics))
 }
-
-private fun getRc4(key: ByteArray): Cipher {
-    val rc4 = Cipher.getInstance("RC4")
-    rc4.init(Cipher.ENCRYPT_MODE, SecretKeySpec(key, "RC4"))
-    return rc4
-}
-
-private fun md5Bytes(stringToHash: String): ByteArray {
-    val digest = MessageDigest.getInstance("MD5")
-    return digest.digest(stringToHash.toByteArray())
-}
-
-private fun printHexString(bytes: ByteArray): String {
-    val sb = StringBuilder()
-    for (b in bytes) {
-        sb.append(String.format("%02x", b))
-    }
-    return sb.toString()
-}

src/main/kotlin/mdnet/base/web/ImageServer.kt

@@ -0,0 +1,215 @@
+/* ktlint-disable no-wildcard-imports */
+package mdnet.base.web
+
+import mdnet.base.Constants
+import mdnet.base.Statistics
+import mdnet.cache.CachingInputStream
+import mdnet.cache.DiskLruCache
+import org.apache.http.client.config.CookieSpecs
+import org.apache.http.client.config.RequestConfig
+import org.apache.http.impl.client.HttpClients
+import org.http4k.client.ApacheClient
+import org.http4k.core.*
+import org.http4k.filter.MaxAgeTtl
+import org.http4k.lens.Path
+import org.slf4j.LoggerFactory
+import java.io.BufferedInputStream
+import java.io.BufferedOutputStream
+import java.io.InputStream
+import java.security.MessageDigest
+import java.util.concurrent.Executors
+import java.util.concurrent.atomic.AtomicReference
+import javax.crypto.Cipher
+import javax.crypto.CipherInputStream
+import javax.crypto.CipherOutputStream
+import javax.crypto.spec.SecretKeySpec
+
+private const val THREADS_TO_ALLOCATE = 262144 // 2**18 // Honestly, no reason to not just let 'er rip. Inactive connections will expire on their own :D
+private val LOGGER = LoggerFactory.getLogger(ImageServer::class.java)
+
+class ImageServer(private val cache: DiskLruCache, private val statistics: AtomicReference<Statistics>, private val upstreamUrl: String) {
+    private val executor = Executors.newCachedThreadPool()
+    private val client = ApacheClient(responseBodyMode = BodyMode.Stream, client = HttpClients.custom()
+        .setDefaultRequestConfig(
+            RequestConfig.custom()
+            .setCookieSpec(CookieSpecs.IGNORE_COOKIES)
+            .setConnectTimeout(3000)
+            .setSocketTimeout(3000)
+            .setConnectionRequestTimeout(3000)
+            .build())
+        .setMaxConnTotal(THREADS_TO_ALLOCATE)
+        .setMaxConnPerRoute(THREADS_TO_ALLOCATE)
+        .build())
+
+    fun handler(dataSaver: Boolean, tokenized: Boolean = false): HttpHandler = { request ->
+        val chapterHash = Path.of("chapterHash")(request)
+        val fileName = Path.of("fileName")(request)
+
+        val sanitizedUri = if (dataSaver) {
+            "/data-saver"
+        } else {
+            "/data"
+        } + "/$chapterHash/$fileName"
+
+        if (LOGGER.isInfoEnabled) {
+            LOGGER.info("Request for $sanitizedUri received")
+        }
+        statistics.getAndUpdate {
+            it.copy(requestsServed = it.requestsServed + 1)
+        }
+
+        val rc4Bytes = if (dataSaver) {
+            md5Bytes("saver$chapterHash.$fileName")
+        } else {
+            md5Bytes("$chapterHash.$fileName")
+        }
+        val cacheId = printHexString(rc4Bytes)
+
+        val snapshot = cache.get(cacheId)
+        if (snapshot != null) {
+            request.handleCacheHit(sanitizedUri, getRc4(rc4Bytes), snapshot)
+        } else {
+            request.handleCacheMiss(sanitizedUri, getRc4(rc4Bytes), cacheId)
+        }
+    }
+
+    private fun Request.handleCacheHit(sanitizedUri: String, cipher: Cipher, snapshot: DiskLruCache.Snapshot): Response {
+        // our files never change, so it's safe to use the browser cache
+        return if (this.header("If-Modified-Since") != null) {
+            statistics.getAndUpdate {
+                it.copy(browserCached = it.browserCached + 1)
+            }
+
+            if (LOGGER.isInfoEnabled) {
+                LOGGER.info("Request for $sanitizedUri cached by browser")
+            }
+
+            val lastModified = snapshot.getString(2)
+            snapshot.close()
+
+            Response(Status.NOT_MODIFIED)
+                .header("Last-Modified", lastModified)
+        } else {
+            statistics.getAndUpdate {
+                it.copy(cacheHits = it.cacheHits + 1)
+            }
+
+            if (LOGGER.isInfoEnabled) {
+                LOGGER.info("Request for $sanitizedUri hit cache")
+            }
+
+            respondWithImage(
+                CipherInputStream(BufferedInputStream(snapshot.getInputStream(0)), cipher),
+                snapshot.getLength(0).toString(), snapshot.getString(1), snapshot.getString(2),
+                true
+            )
+        }
+    }
+
+    private fun Request.handleCacheMiss(sanitizedUri: String, cipher: Cipher, cacheId: String): Response {
+        if (LOGGER.isInfoEnabled) {
+            LOGGER.info("Request for $sanitizedUri missed cache")
+        }
+        statistics.getAndUpdate {
+            it.copy(cacheMisses = it.cacheMisses + 1)
+        }
+
+        val mdResponse = client(Request(Method.GET, "$upstreamUrl$sanitizedUri"))
+
+        if (mdResponse.status != Status.OK) {
+            if (LOGGER.isTraceEnabled) {
+                LOGGER.trace("Upstream query for $sanitizedUri errored with status {}", mdResponse.status)
+            }
+            mdResponse.close()
+            return Response(mdResponse.status)
+        }
+
+        if (LOGGER.isTraceEnabled) {
+            LOGGER.trace("Upstream query for $sanitizedUri succeeded")
+        }
+
+        val contentType = mdResponse.header("Content-Type")!!
+        val contentLength = mdResponse.header("Content-Length")
+        val lastModified = mdResponse.header("Last-Modified")
+
+        val editor = cache.edit(cacheId)
+
+        // A null editor means that this file is being written to
+        // concurrently so we skip the cache process
+        return if (editor != null && contentLength != null && lastModified != null) {
+            if (LOGGER.isTraceEnabled) {
+                LOGGER.trace("Request for $sanitizedUri is being cached and served")
+            }
+            editor.setString(1, contentType)
+            editor.setString(2, lastModified)
+
+            val tee = CachingInputStream(
+                mdResponse.body.stream,
+                executor, CipherOutputStream(BufferedOutputStream(editor.newOutputStream(0)), cipher)
+            ) {
+                if (editor.getLength(0) == contentLength.toLong()) {
+                    if (LOGGER.isInfoEnabled) {
+                        LOGGER.info("Cache download for $sanitizedUri committed")
+                    }
+                    editor.commit()
+                } else {
+                    if (LOGGER.isInfoEnabled) {
+                        LOGGER.info("Cache download for $sanitizedUri aborted")
+                    }
+                    editor.abort()
+                }
+            }
+            respondWithImage(tee, contentLength, contentType, lastModified, false)
+        } else {
+            editor?.abort()
+
+            if (LOGGER.isTraceEnabled) {
+                LOGGER.trace("Request for $sanitizedUri is being served")
+            }
+            respondWithImage(mdResponse.body.stream, contentLength, contentType, lastModified, false)
+        }
+    }
+
+    private fun respondWithImage(input: InputStream, length: String?, type: String, lastModified: String?, cached: Boolean): Response =
+        Response(Status.OK)
+            .header("Content-Type", type)
+            .header("X-Content-Type-Options", "nosniff")
+            .header(
+                "Cache-Control",
+                listOf("public", MaxAgeTtl(Constants.MAX_AGE_CACHE).toHeaderValue()).joinToString(", ")
+            )
+            .header("Timing-Allow-Origin", "https://mangadex.org")
+            .let {
+                if (length != null) {
+                    it.body(input, length.toLong()).header("Content-Length", length)
+                } else {
+                    it.body(input).header("Transfer-Encoding", "chunked")
+                }
+            }
+            .let {
+                if (lastModified != null) {
+                    it.header("Last-Modified", lastModified)
+                } else {
+                    it
+                }
+            }.header("X-Cache", if (cached) "HIT" else "MISS")
+}
+
+private fun getRc4(key: ByteArray): Cipher {
+    val rc4 = Cipher.getInstance("RC4")
+    rc4.init(Cipher.ENCRYPT_MODE, SecretKeySpec(key, "RC4"))
+    return rc4
+}
+
+private fun md5Bytes(stringToHash: String): ByteArray {
+    val digest = MessageDigest.getInstance("MD5")
+    return digest.digest(stringToHash.toByteArray())
+}
+
+private fun printHexString(bytes: ByteArray): String {
+    val sb = StringBuilder()
+    for (b in bytes) {
+        sb.append(String.format("%02x", b))
+    }
+    return sb.toString()
+}