Skip to content
This repository has been archived by the owner on Aug 12, 2024. It is now read-only.

[Bug] 500 error when loading select pictures #63

Closed
quintindunn opened this issue Nov 23, 2023 · 1 comment
Closed

[Bug] 500 error when loading select pictures #63

quintindunn opened this issue Nov 23, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@quintindunn
Copy link
Owner

Some pictures (in my experience it has only happened on profile pictures) https://image.production.journal-api.lapse.app/image/upload/q_100//xxxx/xxxx.jpg will throw a error 500, this can be remedied by adjusting the parameters of the API call, for example using:
https://image.production.journal-api.lapse.app/image/upload/,//xxxx/xxxx.jpg doesn't throw an error. I think the best fix would be to default throws parameters to a simple , and then if the user specifies specific parameters adjust it then. It also might be possible to default it to q_65 which is what the Lapse app uses.

Specifically I found issues with the q parameter, with an image I was testing with I was getting issues when using q_100 but when I bumped it down to q_50 I didn't seem to get any errors.

Before fixing this I want to perform more tests to try to determine the cause of the issue,

@quintindunn quintindunn added the bug Something isn't working label Nov 23, 2023
@quintindunn
Copy link
Owner Author

Changed default q value to 65 in commit acf906f

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant