Skip to content

Commit

Permalink
Merge pull request #51 from http4s/blocking-requires-blocking
Browse files Browse the repository at this point in the history
Blocking servlets require blocking Servlet I/O
  • Loading branch information
rossabaker authored Jun 6, 2022
2 parents 716a20c + ba3d295 commit 6f5026e
Showing 1 changed file with 9 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -26,14 +26,22 @@ import org.http4s.server._
import javax.servlet.http.HttpServletRequest
import javax.servlet.http.HttpServletResponse

class BlockingHttp4sServlet[F[_]](
class BlockingHttp4sServlet[F[_]] private (
service: HttpApp[F],
servletIo: ServletIo[F],
serviceErrorHandler: ServiceErrorHandler[F],
dispatcher: Dispatcher[F],
)(implicit F: Sync[F])
extends Http4sServlet[F](service, servletIo, dispatcher) {

def this(
service: HttpApp[F],
servletIo: BlockingServletIo[F],
serviceErrorHandler: ServiceErrorHandler[F],
dispatcher: Dispatcher[F],
)(implicit F: Sync[F]) =
this(service, servletIo: ServletIo[F], serviceErrorHandler, dispatcher)(F)

@deprecated("Binary compatibility", "0.23.12")
private[servlet] def this(
service: HttpApp[F],
Expand Down

0 comments on commit 6f5026e

Please sign in to comment.