lezhnev74 / openapi-psr7-validator

It validates PSR-7 messages (HTTP request/response) against OpenAPI specifications
https://lessthan12ms.com/openapi-with-php-documenting-and-testing-api-automatically/
MIT License
169 stars 10 forks source link

Collaboration #65

Closed philsturgeon closed 5 years ago

philsturgeon commented 5 years ago

Hey there! I think there is a great chance for you folks to collaborate here. I am opening up the conversation here as it has the most stars, but it could be anywhere. They are all good implementations.

These packages all do the same thing.

  1. Read a JSON or YAML file
  2. Allow general PSR-7/PSR-15 middleware to be created for this file
  3. Offer at least request validation (maybe response too)
  4. Offer errors in various formats (RFC 7807 or others)
  5. Support custom formats

I think you could all team up on a new @thephpleague project. I bet they'd take you if you all worked together, I can provide some guidance if needed, and then you just archive things on GitHub, mark them abandoned on Packagist, and the mega-package lives on forever despite whatever hurdles life throw at you individually.

LMK if you like the sound of this. OpenAPI can be just too much for single maintainers to handle solo. 🤷‍♂

lezhnev74 commented 5 years ago

It is a smart move to have a single project for this type of problems run by the community (instead of pouring efforts in many projects). The league seems to be a perfect place for that kind of activity. This package is licensed under MIT so there are no problems to use it for starting a project on the league foundation. I am not archiving this, but if such transition happens I'll add a banner to this repo that a community edition (which is better) is available on the league repo.

mieszkomalawski commented 5 years ago

+1 to this

Having one quality package under league umbrella sounds great. BTW I will have a presentation this week about OpenAPI and validation at https://summit.phpers.pl/pl/

hkarlstrom commented 5 years ago

+1

philsturgeon commented 5 years ago

This is great! I poked my friend Frank De Jonge who is still active with the league:

Sounds like a good fit. I'll start off the conversation on this end. In the meanwhile, if they can join the slack channel, that'd speed thing up and make it easy to communicate: slack.thephpleague.com

cebe commented 5 years ago

These packages all do the same thing.

  1. Read a JSON or YAML file

There is also duplicated effort here:

Happy to participate in the collaboration.

brayniverse commented 5 years ago

You're all great! I'm so happy to see you all talking about collaborating.

My team and I were investigating ways to test API results against our API spec and had to downgrade from OpenAPI 3 to Swagger 2 because OpenAPI 3 wasn't as widely supported. I can't remember which implementation we landed on, but I'd be excited to migrate to a common implementation that all you bright developers are contributing to. I'd love to help however I can.

lezhnev74 commented 5 years ago

@hkarlstrom what's your email, just want to get in touch about this project?

lezhnev74 commented 5 years ago

image

lezhnev74 commented 5 years ago

Thank you guys! It has happened. Go to https://github.com/thephpleague/openapi-psr7-validator

scaytrase commented 5 years ago

Congrats! Do we have any roadmap? Also can we transfer current significant issues to the new project?

lezhnev74 commented 5 years ago

Yes, I did migrate some of them. Feel free to add the rest that you feel needs to be done.

philsturgeon commented 5 years ago

This is amazing, thank you @lezhnev74, and thank you @mieszkomalawski and @hkarlstrom for all working together on this! I can't wait to see one awesome super validator be the best of all three implementations.

philsturgeon commented 5 years ago

Btw @hkarlstrom could you move some of your feature requests over?

https://github.com/hkarlstrom/openapi-validation-middleware/issues/33 https://github.com/hkarlstrom/openapi-validation-middleware/issues/27

hkarlstrom commented 5 years ago

Sure, will do it tmr!

On Thu, 19 Sep 2019 at 17.39, Phil Sturgeon notifications@github.com wrote:

Btw @hkarlstrom https://github.com/hkarlstrom could you move some of your feature requests over?

hkarlstrom/openapi-validation-middleware#33 https://github.com/hkarlstrom/openapi-validation-middleware/issues/33 hkarlstrom/openapi-validation-middleware#27 https://github.com/hkarlstrom/openapi-validation-middleware/issues/27

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/lezhnev74/openapi-psr7-validator/issues/65?email_source=notifications&email_token=AAGBTZZ5BRRYKASHMIURZWDQKOFK3A5CNFSM4ITGP7ZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7DWQII#issuecomment-533162017, or mute the thread https://github.com/notifications/unsubscribe-auth/AAGBTZ2IUH6RNWJADH6MCCDQKOFK3ANCNFSM4ITGP7ZA .

scaytrase commented 5 years ago

I think we should also mark packages (https://packagist.org/packages/lezhnev74/openapi-psr7-validator) as abandoned with suggested replacement in order to make users transit to new one