Skip to content

Commit

Permalink
removing auxServerMaxStream because it is not suitable for HTTP/3
Browse files Browse the repository at this point in the history
  • Loading branch information
kazu-yamamoto committed Nov 24, 2023
1 parent ab20537 commit 4b5d938
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 6 deletions.
1 change: 0 additions & 1 deletion Network/HTTP2/Client.hs
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,6 @@ module Network.HTTP2.Client (
-- * Aux
Aux,
auxPossibleClientStreams,
auxServerMaxStreams,

-- * Types
Method,
Expand Down
3 changes: 3 additions & 0 deletions Network/HTTP2/Client/Internal.hs
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
module Network.HTTP2.Client.Internal (
Request (..),
Response (..),
ClientConfig (..),
Settings (..),
Aux (..),

-- * Low level
Stream,
Expand Down
1 change: 0 additions & 1 deletion Network/HTTP2/Client/Run.hs
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,6 @@ run cconf@ClientConfig{..} conf client = do
aux ctx =
Aux
{ auxPossibleClientStreams = possibleClientStream ctx
, auxServerMaxStreams = serverMaxStreams ctx
}
clientCore ctx mgr req processResponse = do
strm <- sendRequest ctx mgr scheme authority req
Expand Down
4 changes: 0 additions & 4 deletions Network/HTTP2/Client/Types.hs
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,4 @@ newtype Response = Response InpObj deriving (Show)
data Aux = Aux
{ auxPossibleClientStreams :: IO Int
-- ^ How many streams can be created without blocking.
, auxServerMaxStreams :: IO Int
-- ^ Getting server's SETTINGS_MAX_CONCURRENT_STREAMS.
-- If the server does not inform it,
-- 'maxBound' is used.
}

0 comments on commit 4b5d938

Please sign in to comment.