-
Notifications
You must be signed in to change notification settings - Fork 36
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
Any way to set protocol of url2png_image_url ? #21
Comments
Have you already tried? The protocol from the requested page should be used. More details can be found in this commit: 2601062 |
Strange, I'm getting this error: Mixed Content: The page at 'https://getdev-staging.herokuapp.com/users/31' was loaded over HTTPS, but requested an insecure image 'http://api.url2png.com/v6/P289C47C4185BF3/8d921085e9c7df5ccbbd51447ac99be4/png/?url=http%3A%2F%2Fwww.jjsuspenders.com'. This content should also be served over HTTPS. Here's my ERB code: <% url = url2png_image_url project.url %> |
Same problem. This gem probably needs a new maintainer. I'm happy to take over fixing if @hoetmaaiers can give me push access or would point the official repo to https://github.com/polleverywhere/url2png-gem. |
Hi @bradgessler, I prefer to first review and accept some pull requests. This might convince me to hand over the ownership of the gem to you. I'll be glad to see the first PR's coming |
I ran into this again today, tried to use this gem, forgot I opened this issue, and created I'm still open to maintaining this gem, but only if you're open to replacing this source code with the |
I see no issue in both gems being available? Replacing this gem with url2png-plus will break existing gems because they expect a different public API. Referring to url2png-plus is a possibility I guess. |
I’d agree there’s no issue. It’s more a question of ambiguity, but it’s not that bad.
For the sake of discussion if we went through with this it could be an exercise or keeping the helpers in this gem to minimize breakage, signify breaking API changes with SemVer, and ship.
…On Nov 11, 2017, 11:08 -0800, Hoetmaaiers ***@***.***>, wrote:
I see no issue in both gems being available? Replacing this gem with url2png-plus will break existing gems because they expect a different public API. Referring to url2png-plus is a possibility I guess.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
I am not sure what you are suggesting... |
I have an https page, I'm trying to set url2png_image_url protocol to https. Is there any option to do this?
Thanks!
The text was updated successfully, but these errors were encountered: