mirror of
https://github.com/flyimg/flyimg.git
synced 2026-04-25 01:35:56 +03:00
[GH-ISSUE #191] Image path parameters is ignored #690
Labels
No labels
Docs
Docs
Docs
Security
UnitTest
bug
dependencies
duplicate
enhancement
enhancement
enhancement
hacktoberfest
help wanted
invalid
pull-request
question
stale
version 1
version 2
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/flyimg#690
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @yoldar on GitHub (Sep 13, 2018).
Original GitHub issue: https://github.com/flyimg/flyimg/issues/191
Originally assigned to: @sadok-f on GitHub.
Hi guys!
Is that possible to not ignore image path parameters?
When I try this in container:
curl "http://localhost/upload/w_500,h_500,q_90/http://some.com/uploads/image.jpg?token=342342342334&someParam=dlklksfdldfI got error:
Download image in container with regular wget works normal.
P.s. I really need this because downloading images can only be with access params like token and so on.
@sadok-f commented on GitHub (Sep 13, 2018):
Hi @yoldar !
The problem is with
?which will convert the token into a parameter GET in the flyimg application, maybe if you try to encode it before pass it to flyimg server:curl "http://localhost/upload/w_500,h_500,q_90/http://some.com/uploads/image.jpg%3Ftoken=342342342334@yoldar commented on GitHub (Sep 14, 2018):
Thanks!
@yoldar commented on GitHub (Sep 14, 2018):
@sadok-f
Could you please advice is that possible to ignore all parameters when making hash naming for original and optimized files? I mean original-HASH than stored under "tmp" folder and HASH.webp under "uploads" folder?
P.s. Because access token changes every time but image still the same.
@sadok-f commented on GitHub (Sep 14, 2018):
@yoldar Thanks for pointing this out, in fact, the original image should be saved without considering any extras arguments, so the hash should be generated only with the image source URL.
I'll work on it this weekend and hopefully submit a PR regarding that!
Thank you again!
@yoldar commented on GitHub (Sep 16, 2018):
Thanks! It would be great!
P.s. Open source rulez! 😁
@sadok-f commented on GitHub (Sep 16, 2018):
Here we go