refactor(users/Profpatsch/whatcd-resolver): continue http
Ideally there’d be a better generic abstraction of doing basic http calls (with tracing) in the future, but for now just reexport. Change-Id: Id7548739ea62e9172f2773f8db79fe726096b7f1 Reviewed-on: https://cl.tvl.fyi/c/depot/+/12136 Tested-by: BuildkiteCI Reviewed-by: Profpatsch <mail@profpatsch.de>
This commit is contained in:
parent
8908fd18ca
commit
37b2c2ce78
3 changed files with 7 additions and 6 deletions
|
@ -5,11 +5,15 @@ module Http
|
|||
RequestOptions (..),
|
||||
mkRequestOptions,
|
||||
httpJson,
|
||||
Http.httpBS,
|
||||
Http.Request,
|
||||
Http.setRequestMethod,
|
||||
Http.setQueryString,
|
||||
Http.setRequestBodyLBS,
|
||||
Http.setRequestHeader,
|
||||
Http.getResponseStatus,
|
||||
Http.getResponseHeader,
|
||||
Http.getResponseHeaders,
|
||||
Http.getResponseBody,
|
||||
)
|
||||
where
|
||||
|
|
|
@ -16,7 +16,6 @@ import IHP.HSX.QQ (hsx)
|
|||
import Json qualified
|
||||
import Label
|
||||
import MyPrelude
|
||||
import Network.HTTP.Client.Conduit qualified as Http
|
||||
import Network.HTTP.Types.URI qualified as Url
|
||||
import Network.URI (URI)
|
||||
import Optional
|
||||
|
|
|
@ -19,8 +19,6 @@ import Http qualified
|
|||
import Json qualified
|
||||
import Label
|
||||
import MyPrelude
|
||||
import Network.HTTP.Client.Conduit qualified as Http
|
||||
import Network.HTTP.Simple qualified as Http
|
||||
import Network.HTTP.Types
|
||||
import Network.Wai.Parse qualified as Wai
|
||||
import OpenTelemetry.Trace qualified as Otel hiding (getTracer, inSpan, inSpan')
|
||||
|
@ -503,17 +501,17 @@ httpTorrent span req =
|
|||
>>= assertM
|
||||
span
|
||||
( \resp -> do
|
||||
let statusCode = resp & Http.responseStatus & (.statusCode)
|
||||
let statusCode = resp & Http.getResponseStatus & (.statusCode)
|
||||
contentType =
|
||||
resp
|
||||
& Http.responseHeaders
|
||||
& Http.getResponseHeaders
|
||||
& List.lookup "content-type"
|
||||
<&> Wai.parseContentType
|
||||
<&> (\(ct, _mimeAttributes) -> ct)
|
||||
if
|
||||
| statusCode == 200,
|
||||
Just "application/x-bittorrent" <- contentType ->
|
||||
Right $ (resp & Http.responseBody)
|
||||
Right $ (resp & Http.getResponseBody)
|
||||
| statusCode == 200,
|
||||
Just otherType <- contentType ->
|
||||
Left [fmt|Redacted returned a non-torrent body, with content-type "{otherType}"|]
|
||||
|
|
Loading…
Reference in a new issue