Skip to content

Commit

Permalink
refactor: replaced Got HTTP client with native fetch (#34)
Browse files Browse the repository at this point in the history
  • Loading branch information
TedCha authored Sep 21, 2024
1 parent 6082cee commit 5fbeff2
Show file tree
Hide file tree
Showing 6 changed files with 345 additions and 255 deletions.
3 changes: 3 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -195,6 +195,9 @@ Out-of-order Loki errors can occur due to the asynchronous nature of Pino. The f
## Dropped logs
If any network issues occur, the logs can be dropped. The recommendation is therefore to implement a failover solution, this will vary greatly from system to system.

## Node v18+ Required
As the pino-loki library uses the native Node fetch, any consumer must be using a version of Node greater than v18.0.0.

## Developing

### Requirements
Expand Down
3 changes: 1 addition & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@
},
"dependencies": {
"commander": "^12.0.0",
"got": "^11.8.6",
"pino-abstract-transport": "^1.2.0",
"pump": "^3.0.0"
},
Expand All @@ -74,7 +73,7 @@
"c8": "^9.1.0",
"dotenv": "^16.4.5",
"eslint": "^8.57.0",
"nock": "^13.5.4",
"msw": "^2.4.9",
"pino": "^9.0.0",
"pino-pretty": "^11.0.0",
"prettier": "^3.2.5",
Expand Down
Loading

0 comments on commit 5fbeff2

Please sign in to comment.