Skip to content
This repository has been archived by the owner on Apr 9, 2022. It is now read-only.

Move getKey() from request query parameters to body #313

Open
alan-fang opened this issue Jul 25, 2019 · 1 comment
Open

Move getKey() from request query parameters to body #313

alan-fang opened this issue Jul 25, 2019 · 1 comment
Labels

Comments

@alan-fang
Copy link
Contributor

This https://stackoverflow.com/questions/25385559/rest-api-best-practices-args-in-query-string-vs-in-request-body suggests that key should not be passed as a query parameter, but rather in the request body.

@alan-fang alan-fang added good first issue Good for newcomers invalid This doesn't seem right Priority: Medium labels Jul 25, 2019
@alan-fang alan-fang removed the invalid This doesn't seem right label Aug 6, 2019
@tko22
Copy link
Member

tko22 commented Aug 15, 2019

i think it would depend on the authentication server stuff. Most likely, from what I remember, it would be in the header under the Authorization: Bearer header

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

No branches or pull requests

2 participants