Open idanw206 opened 6 years ago
I signed it!
CLAs look good, thanks!
@theganyo any plan to roll this out? thanks!
@theganyo can you please respond? thanks!
There are tons of fixes in Sway 2.0 and some relate to array type which really required. I also would appreciate this release as well. Thanks
Same here. Really need the fixes for arrays with a single element only. Please update the lib.
Can any of you check this out? @earth2marsh @gbrail @theganyo @whitlockjc @WWitman
If you are really in need for the latest pull requests you can use my fork and I have already merged the pull requests which I am using it in my production: https://github.com/multivacplatform/swagger-node-runner
Use it like this:
"swagger-express-runner": "git@github.com:multivacplatform/swagger-node-runner.git",
Or you can fork it yourself and do the changes. Either way, when the maintainer comes back you can switch to the original repo with no problem. Hope this helps.
@theganyo Can we get this in? This seems to solve CVE-2018-3750 @earth2marsh @gbrail @whitlockjc @WWitman Anyone?
Bump! @theganyo,@earth2marsh, @gbrail, @whitlockjc, @WWitman
I went ahead and created npm packages to solve this. https://www.npmjs.com/package/swagger-express-mw1 https://www.npmjs.com/package/swagger-node-runner1
These are based on forked repo from https://github.com/multivacplatform/swagger-node-runner Thanks to @maziyarpanahi
We can use these for now and wait for apigee-127 to fix the original package
Great! Many thanks @adityaguru149 👍
bump @theganyo,@earth2marsh, @gbrail, @whitlockjc, @WWitman
Do the contributors of this project need help? This has been ignored for a while - I can help if anything is needed as I would like this fix as well. AKA bump
Since I've looked at this thread more than once, I figured I would help future people out, please see: How to migrate a Node JS Express project out of Swagger-Express
Note: I am not associated with either project but just wanted to share a migration guide.
Any chance to see this merged? This would really help a lot of people, and would likely require no changes in swagger-node-runner 🤔 cc @theganyo
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g.
I signed it!
) and we'll verify it.What to do if you already signed the CLA
Individual signers
Corporate signers