-
Notifications
You must be signed in to change notification settings - Fork 102
Add filepicker_image_url to work with policies #95
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
Conversation
Sure! I'll test, sorry for the delay. |
Thanks for working on this btw :) So I tried it out and it didn't work. You can see an example here: http://dstock.herokuapp.com/i/12 I have my keys setup here:
Then to display the image I have:
|
When I remove the [uuid=AF5177F230ED4DC4] This action has been secured by the developer of this website. Error: The specified policy does not allow the call read |
Call parameter should be specified when you are creating your policies. It's same kind of parameter as expiry or handle in your JSON policy. For example: {"handle":"KW9EJhYtS6y48Whm2S6D", "expiry": 1508141504, "call":["pick","read","convert"]} |
Does that help? |
@bcackerman in this jsfiddle we have a example using the JS api. I will show some tests using this params. Without policy: https://www.filepicker.io/api/file/KW9EJhYtS6y48Whm2S6D Got the signature error message. Without convertion and with policy: With convertion and with policy: As you can see in both examples the only thing we need to do is to append the policy and signature params on URL. And this is tested here Could you try this again? You pointed to the right branch? As is showed your link on the helper when using this branch and with secret key? |
So strange, in my In my gemfile I have Images still broken when I turn on security from Filepicker.io |
Still using this tag Here's an example image url When I remove the convert params and go to |
@bcackerman, check out #104, which resolved this for us. |
Closed since it is continued on #105 |
Closes #93
@bcackerman do you can test the policies with this branch? As you can see now it is appended the signature and policy to query as documented here.
Please test this with a convertions and without a convertion.