GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Define http client connection buffer size · Issue #842 · cpp-netlib/cpp-netlib

Define http client connection buffer size #842

New issue
Copy link
New issue
Copy link
Closed
Closed
Define http client connection buffer size#842
Copy link

Description

@umennel
umennel
opened on May 26, 2018
Issue body actions

It is possible to define the server connection buffer size using BOOST_NETWORK_HTTP_SERVER_CONNECTION_BUFFER_SIZE. It is set to a default value if undefined. See boost/network/protocol/http/server/async_connection.hpp. Depending on the hardware, this may significantly increase data transfer rates, as shown in #726.
Could we have a similar macro for the client connection?

Activity

  1. deanberris commented on May 26, 2018

    @deanberris
    deanberris
    on May 26, 2018
    Member
    More actions

    Sounds like a good idea to me. I do think that since the client has options you can set to affect this already -- if not, then we should do both, controlling the size of the buffers from options and potentially changing the default for the option with a compile-time macro.

    Do you mind sending a pull request to do that?

  2. umennel commented on May 27, 2018

    @umennel
    umennel
    on May 27, 2018
    Author
    More actions

    @deanberris, I didn't find a way to use the options because the buffer size is set at compile-time. See #843

  3. deanberris
    added a commit that references this issue on May 28, 2018

    Make http client connection buffer size configurable (#843)

    This issue will close once commit 5a1c841 is merged into the 'main' branch.
    5a1c841
  4. deanberris
    closed this as completedon May 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      You are visitor 01402503 — GeezerVison does not store the pages you look up.
      Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.