Skip to content
This repository has been archived by the owner on May 28, 2024. It is now read-only.

Smart Home: Allow to set https agent options #422

Open
bramkragten opened this issue Apr 12, 2021 · 1 comment
Open

Smart Home: Allow to set https agent options #422

bramkragten opened this issue Apr 12, 2021 · 1 comment

Comments

@bramkragten
Copy link

bramkragten commented Apr 12, 2021

We would like to set maxSockets and other options for the https agent, it would be nice if we could pass an agent to use:

const agent = new https.Agent({
   maxSockets: 25
});

const sh = smarthome({ jwt, agent });

It would probably also be wise to set a sane default maxSockets, as the default infinity might not be wise for an app dealing with a lot of traffic,

I can make a PR for this if accepted.

@Fleker
Copy link
Member

Fleker commented Apr 13, 2021

FYI @proppy

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants