Added support to sway 2 by bumping dep version and fixing tests#139
Added support to sway 2 by bumping dep version and fixing tests#139idanw206 wants to merge 2 commits intoapigee-127:masterfrom
Conversation
|
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). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here (e.g. What to do if you already signed the CLAIndividual signers
Corporate signers
|
|
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. |
|
Same here. Really need the fixes for arrays with a single element only. Please update the lib. |
|
Can any of you check this out? |
|
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: Use it like this: 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. |
|
@theganyo Can we get this in? This seems to solve CVE-2018-3750 |
|
Bump! @theganyo,@earth2marsh, @gbrail, @whitlockjc, @WWitman |
|
I went ahead and created npm packages to solve this. These are based on forked repo from https://github.com/multivacplatform/swagger-node-runner 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: 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 |
No description provided.