Fixing scope (#7)

* fixing scope to support multiple registrations
* adding github actions

Author: jkyberneees

.github/FUNDING.yml

@@ -0,0 +1,3 @@
+# These are supported funding model platforms
+
+custom: https://www.paypal.me/kyberneees

.github/ISSUE_TEMPLATE/bug_report.md

@@ -0,0 +1,38 @@
+---
+name: Bug report
+about: Create a report to help us improve
+title: ''
+labels: ''
+assignees: ''
+
+---
+
+**Describe the bug**  
+A clear and concise description of what the bug is.
+
+**To Reproduce**  
+Steps to reproduce the behavior:
+1. Go to '...'
+2. Click on '....'
+3. Scroll down to '....'
+4. See error
+
+**Expected behavior**  
+A clear and concise description of what you expected to happen.
+
+**Screenshots**  
+If applicable, add screenshots to help explain your problem.
+
+**Desktop (please complete the following information):**  
+ - OS: [e.g. iOS]
+ - Browser [e.g. chrome, safari]
+ - Version [e.g. 22]
+
+**Smartphone (please complete the following information):**  
+ - Device: [e.g. iPhone6]
+ - OS: [e.g. iOS8.1]
+ - Browser [e.g. stock browser, safari]
+ - Version [e.g. 22]
+
+**Additional context**  
+Add any other context about the problem here.

.github/ISSUE_TEMPLATE/custom.md

@@ -0,0 +1,10 @@
+---
+name: Custom issue template
+about: Describe this issue template's purpose here.
+title: ''
+labels: ''
+assignees: ''
+
+---
+
+

.github/ISSUE_TEMPLATE/feature_request.md

@@ -0,0 +1,24 @@
+---
+name: Feature request
+about: Suggest an idea for this project
+title: ''
+labels: ''
+assignees: ''
+
+---
+
+**Have you contributed with at least a little donation to support the development of this project?**
+- Paypal: https://www.paypal.me/kyberneees
+- [TRON](https://www.binance.com/en/buy-TRON) Wallet: `TJ5Bbf9v4kpptnRsePXYDvnYcYrS5Tyxus`
+
+**Is your feature request related to a problem? Please describe.**  
+A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
+
+**Describe the solution you'd like**  
+A clear and concise description of what you want to happen.
+
+**Describe alternatives you've considered**  
+A clear and concise description of any alternative solutions or features you've considered.
+
+**Additional context**  
+Add any other context or screenshots about the feature request here.

.github/workflows/tests.yaml

@@ -0,0 +1,21 @@
+name: tests
+on: [push, pull_request]
+
+jobs:
+  testing:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v1
+      - name: Setup Environment (Using NodeJS 16.x)
+        uses: actions/setup-node@v1
+        with:
+          node-version: 16.x
+
+      - name: Install dependencies
+        run: npm install
+
+      - name: Linting
+        run: npx standard
+
+      - name: Run tests
+        run: npm run test

index.js

@@ -1,58 +1,58 @@
 module.exports = (res, cb) => {
-  res._parent = {
+  const parent = {
     write: res.write,
     end: res.end,
     content: undefined,
     ended: false
   }
 
   res.write = function (content) {
-    accumulate(res, content)
-    return res._parent.write.apply(res, arguments)
+    accumulate(parent, content)
+    return parent.write.apply(res, arguments)
   }
 
   res.end = function (content, encoding) {
-    if (!res._parent.ended) {
-      res._parent.ended = true
+    if (!parent.ended) {
+      parent.ended = true
 
-      accumulate(res, content)
+      accumulate(parent, content)
       const headers = res.getHeaders()
-      const payload = map(res.statusCode, headers, res._parent.content, encoding)
+      const payload = map(res.statusCode, headers, parent.content, encoding)
 
       setImmediate(() => {
         cb(payload)
       })
     }
 
-    return res._parent.end.apply(res, arguments)
+    return parent.end.apply(res, arguments)
   }
 }
 
 function map (status, headers, data, encoding) {
   return {
-    status: status,
-    headers: headers,
-    data: data,
+    status,
+    headers,
+    data,
     encoding: typeof encoding === 'string' ? encoding : null
   }
 }
 
-function accumulate (res, content) {
+function accumulate (parent, content) {
   if (content) {
     if (typeof content === 'string') {
-      res._parent.content = (res._parent.content || '') + content
+      parent.content = (parent.content || '') + content
     } else if (Buffer.isBuffer(content)) {
-      let oldContent = res._parent.content
+      let oldContent = parent.content
 
       if (typeof oldContent === 'string') {
         oldContent = Buffer.from(oldContent)
       } else if (!oldContent) {
         oldContent = Buffer.alloc(0)
       }
 
-      res._parent.content = Buffer.concat([oldContent, content], oldContent.length + content.length)
+      parent.content = Buffer.concat([oldContent, content], oldContent.length + content.length)
     } else {
-      res._parent.content = content
+      parent.content = content
     }
   }
 }

package.json

@@ -4,6 +4,7 @@
   "description": "Allows to capture HTTP response content and headers on request end.",
   "main": "index.js",
   "scripts": {
+    "format": "standard --fix",
     "test": "nyc mocha test/*.test.js --exit"
   },
   "repository": {
@@ -25,7 +26,6 @@
   "devDependencies": {
     "chai": "^4.3.7",
     "mocha": "^10.2.0",
-    "nyc": "^15.1.0",
-    "standard": "^17.0.0"
+    "nyc": "^15.1.0"
   }
 }

test/smoke.test.js

@@ -39,6 +39,28 @@ describe('on-http-end', () => {
     res.end('world')
   })
 
+  it('should support multiple callback registrations', function (done) {
+    onFinished(res, (payload) => {
+      expect(payload.data).to.equal('hello world')
+      expect(data).to.equal('hello world')
+    })
+    onFinished(res, (payload) => {
+      expect(payload.data).to.equal('hello world')
+      expect(data).to.equal('hello world')
+    })
+    onFinished(res, (payload) => {
+      expect(payload.data).to.equal('hello world')
+      expect(data).to.equal('hello world')
+      done()
+    })
+
+    res.write(undefined)
+    res.write('h')
+    res.write(Buffer.from('ello'))
+    res.write(' ')
+    res.end('world')
+  })
+
   it('should accumulate content and encoding', function (done) {
     onFinished(res, (payload) => {
       expect(payload.encoding).to.equal('utf-8')