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

respect form hash option on incoming octect/stream requests, enabling… #407

Closed

Conversation

masterkain
Copy link
Contributor

… hash calculation

@felixge
Copy link
Collaborator

felixge commented Jan 11, 2017

See #412

@tunnckoCore
Copy link
Member

Fixed in #545 branch

@tunnckoCore tunnckoCore reopened this Jan 29, 2020
tunnckoCore added a commit that referenced this pull request Jan 29, 2020
close #407

Co-authored-by: Claudio Poli <[email protected]>
Signed-off-by: Charlike Mike Reagent <[email protected]>
tunnckoCore added a commit that referenced this pull request Feb 12, 2020
* feat: introduce Plugins API, fix tests
* chore: update changelog
* chore: lgtm tweaks?
* start plugins tests; update exports
* chore(custom-plugins-tests): make it really passing
* chore: more tests; add nyc test coverage
* chore: move to GitHub Actions CI
* fix: respect form hash option on incoming octect/stream requests

close #407

* fix: tests, use dezalgo and once for parse callback

- it is recommended plugins to return the this/self/formidable instance
- add 2 deps: `once` and `dezalgo` for ensurance of the `cb` param of .parse()
  + otherwise it COULD (and happened already few times) be called multiple times
- update unit tests to use Jest

* chore: tweaks, publish `dev` dist-tag (with Plugins API)
* chore: sync & tweak
* fix: check strictly for multipart/form-data in multipart parser
* chore: publish dev.2
* fix: tests updates
* fix: it seems we support multipart/related too
* chore: download badges
* Update package.json
* chore: sync

Signed-off-by: Charlike Mike Reagent <[email protected]>
Co-authored-by: Claudio Poli <[email protected]>
@tunnckoCore
Copy link
Member

@all-contributors please add @masterkain for code bugfix

@allcontributors
Copy link
Contributor

@tunnckoCore

I've put up a pull request to add @masterkain! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants