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.

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

docs: Check TS in docs using typescript-docs-verifier #1505

Merged
merged 21 commits into from
Jul 4, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
5bf5624
docs: added aegir doc-check command to compile Typescript in document…
maschad Jan 13, 2023
a351954
doc: updated limits doc according to ts doc verifier errors (wip)
maschad Jan 13, 2023
61cc783
Merge branch 'master' into docs/check-ts-readme
achingbrain Jan 17, 2023
62706de
Merge branch 'master' into docs/check-ts-readme
maschad Apr 21, 2023
0ad1323
docs: updated linting for ts on docs (#1228)
maschad Apr 21, 2023
e5ab618
Merge branch 'master' into docs/check-ts-readme
maschad Apr 24, 2023
bbbf330
Merge branch 'master' into docs/check-ts-readme
maschad Apr 25, 2023
f3c95ce
Merge branch 'master' into docs/check-ts-readme
maschad May 1, 2023
8a4b309
docs: update metrics documentation (#1228)
maschad May 3, 2023
924e95c
doc: linting fixes (#1228)
maschad May 3, 2023
999575a
docs: move config to aegir repo (#1228)
maschad May 3, 2023
21a7087
Merge branch 'master' into docs/check-ts-readme
maschad May 29, 2023
f485990
deps: upgraded aegir to 39.0.9 to use latest doc-check (#1228)
maschad May 29, 2023
f9a5840
docs: ignore metrics dependency as it is needed for docs (#1228)
maschad May 29, 2023
9a43868
Merge branch 'master' into docs/check-ts-readme
maschad Jun 21, 2023
2b42331
docs: added doc check script (#1228)
maschad Jun 21, 2023
bb75005
Merge branch 'master' into docs/check-ts-readme
maschad Jun 22, 2023
ba36b95
Merge branch 'master' into docs/check-ts-readme
maschad Jun 28, 2023
ef3e569
docs: added doc-check to doc workspace (#1228)
maschad Jun 28, 2023
0948c94
docs: added doc-check to doc workspace (#1228)
maschad Jun 28, 2023
ec3ed56
Add private flag to avoid publishing to npm
maschad Jul 4, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ jobs:
- uses: ipfs/aegir/actions/cache-node-modules@master
- run: npm run --if-present lint
- run: npm run --if-present dep-check
- run: npm run --if-present doc-check

test-node:
needs: build
Expand Down
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,8 @@ dist
node_modules
package-lock.json
yarn.lock

# IDE
.DS_STORE
.vscode
packages/transport-webtransport/go-libp2p-webtransport-server/main
8 changes: 4 additions & 4 deletions doc/CONFIGURATION.md
Original file line number Diff line number Diff line change
Expand Up @@ -587,10 +587,10 @@ const node = await createLibp2p({

Libp2p allows you to setup a secure keychain to manage your keys. The keychain configuration object should have the following properties:

| Name | Type | Description |
| --------- | -------- | -------------------------------------------------------------------------------------- |
| pass | `string` | Passphrase to use in the keychain (minimum of 20 characters). |
| dek | `DEKConfig` | the default options for generating the derived encryption key, which, along with the passphrase are input to the PBKDF2 function. For more info see: https://github.com/libp2p/js-libp2p-keychain |
| Name | Type | Description |
| ---- | ----------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| pass | `string` | Passphrase to use in the keychain (minimum of 20 characters). |
| dek | `DEKConfig` | the default options for generating the derived encryption key, which, along with the passphrase are input to the PBKDF2 function. For more info see: https://github.com/libp2p/js-libp2p-keychain |

The keychain will store keys encrypted in the datastore which default is an in memory datastore. If you want to store the keys on disc you need to initialize libp2p with a suitable datastore implementation.

Expand Down
146 changes: 97 additions & 49 deletions doc/LIMITS.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,44 +22,61 @@ This is important for [DoS](https://en.wikipedia.org/wiki/Denial-of-service_atta

It's possible to limit the total amount of connections a node is able to make (combining incoming and outgoing). When this limit is reached and an attempt to open a new connection is made, existing connections may be closed to make room for the new connection (see [Closing connections][#closing-connections]).

* Note: there currently isn't a way to specify different limits for incoming vs. outgoing. Connection limits are applied across both incoming and outgoing connections combined. There is a backlog item for this [here](https://github.com/libp2p/js-libp2p/issues/1508).
- Note: there currently isn't a way to specify different limits for incoming vs. outgoing. Connection limits are applied across both incoming and outgoing connections combined. There is a backlog item for this [here](https://github.com/libp2p/js-libp2p/issues/1508).

We can also limit the number of connections in a "pending" state. These connections have been opened by a remote peer but peer IDs have yet to be exchanged and/or connection encryption and multiplexing negotiated. Once this limit is hit further connections will be closed unless the remote peer has an address in the [allow list](#allowdeny-lists).
We can also limit the number of connections in a 'pending' state. These connections have been opened by a remote peer but peer IDs have yet to be exchanged and/or connection encryption and multiplexing negotiated. Once this limit is hit further connections will be closed unless the remote peer has an address in the [allow list](#allowdeny-lists).

All fields are optional. The default values are defined in [src/connection-manager/index.ts](https://github.com/libp2p/js-libp2p/blob/master/src/connection-manager/index.ts) - please see that file for the current values.

```ts
import { createLibp2p } from 'libp2p'

const node = await createLibp2p({
connectionManager: {
/**
* The total number of connections allowed to be open at one time
*/
maxConnections: number
maxConnections: 100,

/**
* If the number of open connections goes below this number, the node
* will try to connect to randomly selected peers from the peer store
*/
minConnections: number
minConnections: 50,

/**
* How many connections can be open but not yet upgraded
*/
maxIncomingPendingConnections: number
}
})
maxIncomingPendingConnections: 100,
},
});

```

## Closing connections

When choosing connections to close the connection manager sorts the list of connections by the value derived from the tags given to each peer. The values of all tags are summed and connections with lower valued peers are eligible for closing first. If there are tags with equal values, the shortest-lived connection will be closed first.

```js
```ts
import { createLibp2p } from 'libp2p'
import { createEd25519PeerId } from '@libp2p/peer-id-factory'


const libp2p = await createLibp2p({})

const peerId = await createEd25519PeerId()

// tag a peer
await libp2p.peerStore.tagPeer(peerId, 'my-tag', {
value: 50, // 0-100 is the typical value range
ttl: 1000 // optional field, this tag will be deleted after this many ms
})
await libp2p.peerStore.merge(peerId, {
tags: {
'my-tag': {
value: 50, // 0-100 is the typical value range
ttl: 1000, // optional field, this tag will be deleted after this many ms
}
}
});


```

## Inbound connection threshold
Expand All @@ -69,15 +86,19 @@ To prevent individual peers from opening multiple connections to a node, an `inb
All fields are optional. The default values are defined in [src/connection-manager/index.ts](https://github.com/libp2p/js-libp2p/blob/master/src/connection-manager/index.ts) - please see that file for the current values.

```ts
import { createLibp2p } from 'libp2p'

const node = await createLibp2p({
connectionManager: {
/**
* A remote peer may attempt to open up to this many connections per second,
* any more than that will be automatically rejected
*/
inboundConnectionThreshold: number
}
})
inboundConnectionThreshold: 100,
},
});


```

## Stream limits
Expand All @@ -93,37 +114,41 @@ These settings are done on a per-muxer basis, please see the README of the relev
All fields are optional. The default values are defined in [@libp2p/mplex/src/mplex.ts](https://github.com/libp2p/js-libp2p-mplex/blob/master/src/mplex.ts) - please see that file for the current values.

```ts
import { createLibp2p } from 'libp2p'
import { mplex } from '@libp2p/mplex'

const node = await createLibp2p({
muxers: [
streamMuxers: [
mplex({
/**
* The total number of inbound protocol streams that can be opened on a given connection
*/
maxInboundStreams: number
maxInboundStreams: 100,

/**
* The total number of outbound protocol streams that can be opened on a given connection
*/
maxOutboundStreams: number
maxOutboundStreams: 100,

/**
* How much incoming data in bytes to buffer while attempting to parse messages - peers sending many small messages in batches may cause this buffer to grow
*/
maxUnprocessedMessageQueueSize: number
maxUnprocessedMessageQueueSize: 50,

/**
* How much message data in bytes to buffer after parsing - slow stream consumers may cause this buffer to grow
*/
maxStreamBufferSize: number
maxStreamBufferSize: 20,

/**
* Mplex does not support backpressure so to protect ourselves, if `maxInboundStreams` is
* hit and the remote opens more than this many streams per second, close the connection
*/
disconnectThreshold: number
})
]
})
disconnectThreshold: 20,
}),
],
});

```

### Yamux
Expand All @@ -133,25 +158,30 @@ const node = await createLibp2p({
All fields are optional. The default values are defined in [@chainsafe/libp2p-yamux/src/config.ts](https://github.com/ChainSafe/js-libp2p-yamux/blob/master/src/config.ts) - please see that file for the current values.

```ts
import { createLibp2p } from 'libp2p'
import { yamux } from '@chainsafe/libp2p-yamux'

const node = await createLibp2p({
muxers: [
streamMuxers: [
yamux({
/**
* The total number of inbound protocol streams that can be opened on a given connection
*
* This field is optional, the default value is shown
*/
maxInboundStreams: number
maxInboundStreams: 100,

/**
* The total number of outbound protocol streams that can be opened on a given connection
*
* This field is optional, the default value is shown
*/
maxOutboundStreams: number
maxOutboundStreams: 100,
})
]
})
});


```

### Protocol limits
Expand All @@ -165,12 +195,21 @@ Since incoming stream data is buffered until it is consumed, you should attempt
All fields are optional. The default values are defined in [src/registrar.ts](https://github.com/libp2p/js-libp2p/blob/master/src/registrar.ts) - please see that file for the current values.

```ts
libp2p.handle('/my-protocol/1.0.0', (streamData) => {
// ..handle stream
}, {
maxInboundStreams: number
maxOutboundStreams: number
})
import { createLibp2p } from 'libp2p';

const node = await createLibp2p({});

node.handle(
'/my-protocol/1.0.0',
(streamData) => {
// ..handle stream
},
{
maxInboundStreams: 100,
maxOutboundStreams: 100,
}
);

```

## Transport specific limits
Expand All @@ -186,35 +225,42 @@ The [@libp2p/tcp](https://github.com/libp2p/js-libp2p-tcp) transport allows addi
All fields are optional. The full list of options is defined in [@libp2p/tcp/src/index.ts](https://github.com/libp2p/js-libp2p-tcp/blob/master/src/index.ts) - please see that file for more details.

```ts
import { createLibp2p } from 'libp2p'
import {tcp } from '@libp2p/tcp'

const node = await createLibp2p({
transports: [
tcp({
/**
* Inbound connections with no activity in this time frame (ms) will be closed
*/
inboundSocketInactivityTimeout: number
inboundSocketInactivityTimeout: 20,

/**
* Outbound connections with no activity in this time frame (ms) will be closed
*/
outboundSocketInactivityTimeout: number
outboundSocketInactivityTimeout: 20,

/**
* Once this many connections are open on this listener any further connections
* will be rejected - this will have no effect if it is larger than the value
* configured for the ConnectionManager maxConnections parameter
*/
maxConnections: number
})
]
})
maxConnections: 50,
}),
],
});


```

## Allow/deny lists

It is possible to configure some hosts to always accept connections from and some to always reject connections from.

```js
```ts
import { createLibp2p } from 'libp2p'
maschad marked this conversation as resolved.
Show resolved Hide resolved

const node = await createLibp2p({
connectionManager: {
/**
Expand All @@ -233,14 +279,16 @@ const node = await createLibp2p({
* Any connection with a `remoteAddress` property that has any of these
* addresses as a prefix will be immediately rejected
*/
deny: [
deny: [
'/ip4/132.14.52.64/tcp/3984',
'/ip4/234.243.64.2',
'/ip4/34.42',
// etc
]
}
})
],
},
Comment on lines +287 to +288
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These comments should follow the same linting rules as the main codebase; comma-dangle is disallowed.

Suggested change
],
},
]
}

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So the doc-check command which utilizes https://github.com/bbc/typescript-docs-verifier only checks that Typescript code snippets compile, it's not a linter. That could be added in a follow up PR but wasn't the original intention of this.

});
maschad marked this conversation as resolved.
Show resolved Hide resolved


```

## How much memory will be used for buffering?
Expand All @@ -249,10 +297,10 @@ There is no a single config value to control the amount of memory js-libp2p uses

Important details for ascertaining this are:

* Each connection has a multiplexer
* Each multiplexer has a buffer for raw incoming data (`muxer.maxUnprocessedMessageQueueSize`)
* The incoming data is parsed into messages for each stream and queued (`muxer.maxStreamBufferSize`)
* Each multiplexer has a stream limit for number of streams (`muxer.maxInboundStreams`, `muxer.maxOutboundStreams`).
- Each connection has a multiplexer
- Each multiplexer has a buffer for raw incoming data (`muxer.maxUnprocessedMessageQueueSize`)
- The incoming data is parsed into messages for each stream and queued (`muxer.maxStreamBufferSize`)
- Each multiplexer has a stream limit for number of streams (`muxer.maxInboundStreams`, `muxer.maxOutboundStreams`).

As a result, the max amount of memory buffered by libp2p is approximately:

Expand Down
Loading