Skip to content
This repository was archived by the owner on Nov 13, 2021. It is now read-only.

Query String Parameters on a POST Request #138

Open
CarloGarcia opened this issue May 13, 2021 · 4 comments
Open

Query String Parameters on a POST Request #138

CarloGarcia opened this issue May 13, 2021 · 4 comments

Comments

@CarloGarcia
Copy link

Would anyone ever do this? Should this be a "DO NOT" rule, or are there valid situations where this could be allowed.

@tomkerkhove
Copy link
Contributor

I haven't see the need for this, personally.

@gfchaves
Copy link

Hi @CarloGarcia yes indeed I agree with you. This should be a DO NOT rule. POST always should have a defined object contract on message payload and nothing by query string 😎

@CarloGarcia
Copy link
Author

Only exception I can see is if people need to do API versioning using the query string, but best avoided indeed.
For example: https://{myconfig}.azconfig.io/kv?api-version=1.0

@gfchaves
Copy link

gfchaves commented Jun 2, 2021

even with that IMHO shoud be: https://{myconfig}.azconfig.io/v1/kv and not as a query parameter.

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

No branches or pull requests

3 participants