-
-
Notifications
You must be signed in to change notification settings - Fork 137
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
500 on base64 query #3522
Comments
Hi, @Seluj78. This problem seems to be caused by a base64 decode error. Investigating it, I have noticed that the resultant request from typing Something like this, should work. http -v -A bearer -a "$token" GET http://localhost/api/devices\?page\=1\&per_page\=100\&filter\=W3sidHlwZSI6InByb3BlcnR5IiwicGFyYW1zIjp7Im5hbWUiOiJuYW1lIiwib3BlcmF0b3IiOiJjb250YWlucyIsInZhbHVlIjoidCJ9fV0\= |
Right, I just tested again and indeed it seems like I forgot to copy a On the other end though, I think a more explicit message and something other than a 500 would benefit greatly the API users :) |
I strongly agree. We are planning to work on the messages as soon as possible. |
Great, do you want me to open another issue/feature request about this message for you to track it ? |
Perfect. It sounds good to me. |
Description
I was trying to make a filtered query and got a 500 when doing this query
GET https://shellhub.example.com/api/devices?page=1&per_page=100&filter=W3sidHlwZSI6InByb3BlcnR5IiwicGFyYW1zIjp7Im5hbWUiOiJuYW1lIiwib3BlcmF0b3IiOiJjb250YWlucyIsInZhbHVlIjoidCJ9fV0
No clue why, as I just copied the base64 from the UI when searching for the letter
t
in the search barEdition
Community
Version
0.14.1
The text was updated successfully, but these errors were encountered: