Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add configurable connection limit #64

Merged
merged 1 commit into from
Nov 10, 2021
Merged

Add configurable connection limit #64

merged 1 commit into from
Nov 10, 2021

Conversation

justinbot
Copy link
Contributor

Adds support for a new option homeserver.connection_limit which specifies the maximum number of simultaneous HTTP connections to the homeserver.
Defaults to 100, the same value TCPConnector uses if no limit is provided.

Example addition to a config file:

homeserver:
    ...
    # Maximum number of simultaneous HTTP connections to the homeserver.
    connection_limit: 100

@sumnerevans sumnerevans requested a review from tulir November 10, 2021 04:58
sumnerevans
sumnerevans requested changes Nov 10, 2021
View reviewed changes
mautrix/appservice/appservice.py Show resolved Hide resolved
mautrix/appservice/appservice.py Show resolved Hide resolved
@tulir tulir merged commit ac10e4c into mautrix:master Nov 10, 2021
@justinbot justinbot deleted the justinbot/connection-limit branch November 15, 2021 15:22
@justinbot justinbot mentioned this pull request Nov 16, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sumnerevans sumnerevans sumnerevans requested changes

@tulir tulir tulir approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@justinbot @tulir @sumnerevans