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 support for Fastify #246

Draft
wants to merge 19 commits into
base: main
Choose a base branch
from
Draft

Add support for Fastify #246

wants to merge 19 commits into from

Conversation

timokoessler
Copy link
Contributor

@timokoessler timokoessler commented Jun 16, 2024 •
edited
Loading

  • Add sample app
  • Wrap GET requests
  • Wrap other request methods and other functions
  • Patch default module export
  • Add tests
  • Add end2end tests

github-advanced-security[bot]
github-advanced-security bot found potential problems Jun 16, 2024
View reviewed changes
sample-apps/fastify-mysql2/db.js Dismissed Show dismissed Hide dismissed
@codecov Codecov
Copy link

codecov bot commented Jun 20, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 78.36257% with 74 lines in your changes missing coverage. Please review.

Files Patch % Lines
library/sources/Fastify.ts 72.50% 22 Missing ⚠️
library/agent/hooks/WrappableRequireSubject.ts 70.83% 21 Missing ⚠️
library/sources/fastify/wrapRequestHandler.ts 67.44% 14 Missing ⚠️
library/agent/wrapRequire.ts 80.32% 12 Missing ⚠️
library/sources/fastify/contextFromRequest.ts 88.46% 3 Missing ⚠️
library/agent/hooks/ModifyingRequireInterceptor.ts 92.30% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

timokoessler and others added 4 commits June 20, 2024 18:17
@timokoessler
Wrap fastify hooks
05d3622
@hansott
Merge branch 'main' of github.com:AikidoSec/node-RASP into fastify
ae9c981 
* 'main' of github.com:AikidoSec/node-RASP: (42 commits)
  Fix merge
  Add separate xml property to context
  Add tests for sinks
  Update object check and Readme
  Fix comment
  Improve inspectResult test coverage
  Add tests for MethodResultInterceptor
  Add support for fast-xml-parser
  Add new MethodResultInterceptor
  Add additional test and comment
  Try fixing end2end in CI
  Change mysql2 version in test
  Update readme and xml end2end ports
  Fix linting
  Add xml2js end2end tests
  Hono: Parse as text if content type includes xml
  Add xml2js source and sample app
  Fix Hono not checking text bodies
  Add url of express docs
  Add express.subdomains test
  ...
@hansott
regenerate package-lock.json
6397a99
@timokoessler
Prepare wrapping require
63193ed
github-advanced-security[bot]
github-advanced-security bot found potential problems Jun 27, 2024
View reviewed changes
sample-apps/fastify-mysql2/Cats.js Dismissed Show dismissed Hide dismissed
sample-apps/fastify-mysql2/app.js Dismissed Show dismissed Hide dismissed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@timokoessler @hansott