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

Generate both GET and POST endpoints for protocols supporting both #17

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
1 participant
@pimterry
Copy link
Contributor

pimterry commented Apr 4, 2019

AWS services are bad at REST, as it turns out. All services with an 'ec2' or 'query' protocol defined support both GET and POST requests for every operation, which is super weird.

Some evidence to back that up:

It'd be useful to have the specs fully reflect this, so that both formats are described and available. This PR does that, duplicating each endpoint for each methods, only for the specs with the relevant protocol.

@pimterry pimterry force-pushed the pimterry:query-methods branch from dbd6ea6 to 7c048e2 Apr 15, 2019

@pimterry pimterry force-pushed the pimterry:query-methods branch from 7c048e2 to b12fb40 Apr 18, 2019

@pimterry

This comment has been minimized.

Copy link
Contributor Author

pimterry commented Apr 18, 2019

This had a conflict with one of the now-merged PRs. I've rebased & fixed it, should be fine now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.