Http request wrapper build with scalaj-http

httpWrapper.scala

import scalaj.http._


case class httpResponse(code: Int, body: String, url: String)

object httpRequest {
	val options = Map(
		"connTimeout" -> 5000,
		"readTimeout" -> 5000
	)
	
	private def makeOptionList(options: Map[String, Any]): List[HttpOptions.HttpOption] = {
		val requestOptions = this.options ++ options
		List(
			HttpOptions.connTimeout(requestOptions("connTimeout").asInstanceOf[Int]),
			HttpOptions.readTimeout(requestOptions("readTimeout").asInstanceOf[Int])
			)
	}
	
	private def makeResponse(request: Http.Request): httpResponse = {
		if (request.responseCode > 199 && request.responseCode < 400)
			httpResponse(request.responseCode, request.asString, request.getUrl.toString)
		else
			httpResponse(request.responseCode, "", request.getUrl.toString)
	}
	
	def get(url: String, params: Map[String, Any] = Map(), options: Map[String, Any] = Map()): httpResponse = {
		val optionList = makeOptionList(options)
    val requestParams = params map { case (k,v) => (k, v.toString)}
		val request: Http.Request = Http(url).options(optionList).params(requestParams)
		makeResponse(request)
	}
	
	def post(url: String, params: Map[String, Any] = Map(), options: Map[String, Any] = Map()): httpResponse = {
		val optionList = makeOptionList(options)
    val requestParams = params map { case (k,v) => (k, v.toString)}
		val request: Http.Request = Http.post(url).options(optionList).params(requestParams)
		makeResponse(request)
	}
	
}

Each call to .responseCode actually creates a new request. You probably want something like this:

try {
  val (code: Int, header: Map[String, List[String]], body: String) = request.asHeadersAndParse(Http.readString)
  HttpResponse(code, body)
} catch {
  case HttpException(code: Int, message: String, body: String, cause: Throwable) => HttpResponse(code, body)
}