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

TS29520_Nnwdaf_AnalyticsInfo.yaml uses complex types in query parameters with GET #26

Open
adrian-tarau opened this issue May 7, 2020 · 0 comments

Comments

@adrian-tarau
Copy link

I'm a little bit confused about the approach to have a GET request with query parameters as complex types.

Usually, resource and sub-resource identifiers are passed as path params and query are used for filtering, sorting, etc. Most endpoints under this API are actually "searches", with complex types. The recommendation here is to use POST and the query is passed in the body and it can be as complex as it needs to be.

Most clients would have "problems" (there is no default support for complex types in query parameters) and they would have to either generate JSON before setting the parameter or hopefully the client library has support to handle this. For example, Jersey has no default support, but you can plug in an extension point to convert complex types to JSON.

Even if encoding the request is not a problem, you can hit the GET URI limit since the URI will be huge with all those JSON objects...not to mention the horror in the web container logs ;)

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

No branches or pull requests

1 participant