Skip to content

Commit

Permalink
feat: undici fetch in watch (#399)
Browse files Browse the repository at this point in the history
## Description

Change node-fetch for undici fetch and native stream. This is a POC to
determine whether we get better results before changing the whole
node-fetch library out.

_Soak results report 0 watch misses over 6 hours and 45 mins (3 soaks)_

Definition of done:
- [x] Sustains memory in UDS Core 
- [x] Does not miss events in [soak test
](https://github.com/defenseunicorns/pepr/actions/runs/11669014019/job/32490105424)
- [x] still can run `npx pepr dev`


![image](https://github.com/user-attachments/assets/1f040486-4369-47c4-afb3-4c5815af527f)


![image](https://github.com/user-attachments/assets/bce98f4b-edaf-44d3-bd45-c2960608ef63)

## Related Issue

defenseunicorns/pepr#1180

## Type of change

- [ ] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Other (security config, docs update, etc)

## Checklist before merging

- [x] Test, docs, adr added or updated as needed
- [x] [Contributor Guide
Steps](https://docs.pepr.dev/main/contribute/#submitting-a-pull-request)
followed

---------

Signed-off-by: Case Wylie <[email protected]>
  • Loading branch information
cmwylie19 authored Nov 5, 2024
1 parent 8418b43 commit 4123485
Show file tree
Hide file tree
Showing 6 changed files with 264 additions and 616 deletions.
26 changes: 0 additions & 26 deletions .eslintrc.json

This file was deleted.

10 changes: 10 additions & 0 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@
"node-fetch": "2.7.0",
"quicktype-core": "23.0.170",
"type-fest": "4.26.1",
"undici": "6.20.1",
"yargs": "17.7.2"
},
"devDependencies": {
Expand Down
335 changes: 0 additions & 335 deletions src/fluent/http2-watch.test.ts

This file was deleted.

Loading

0 comments on commit 4123485

Please sign in to comment.