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

Download Image in https in Firefox latest broken #5504

Closed
josephzidell opened this Issue Jul 18, 2016 · 3 comments

Comments

Projects
None yet
2 participants
@josephzidell

josephzidell commented Jul 18, 2016

When using https, the download image functionality isn't working. https://jsfiddle.net/highcharts/llexl/

Expected behaviour

The image or pdf downloads.

Actual behaviour

In my application, using v3.0.9, we're getting a Security Warning http://imgur.com/qCCnzEs
Using your standard jsFiddle with the latest master, nothing happens.

Live demo with steps to reproduce

Affected browser(s)

Firefox 47.0.1 Windows 10

My fix

My proposed fix is to change url:"http://export.highcharts.com/" to url:"//export.highcharts.com/" in js/modules/exporting.js

@jon-a-nygaard

This comment has been minimized.

Show comment
Hide comment
@jon-a-nygaard

jon-a-nygaard Jul 19, 2016

Collaborator

@josephzidell Thanks for reporting. While waiting for the maintenance release, which will contain a fix for this issue, you can set the option exporting.url to your desired url, like in this example

Collaborator

jon-a-nygaard commented Jul 19, 2016

@josephzidell Thanks for reporting. While waiting for the maintenance release, which will contain a fix for this issue, you can set the option exporting.url to your desired url, like in this example

@josephzidell

This comment has been minimized.

Show comment
Hide comment
@josephzidell

josephzidell Jul 19, 2016

Thanks for the quick response. We'll use the workaround for now.

josephzidell commented Jul 19, 2016

Thanks for the quick response. We'll use the workaround for now.

@jon-a-nygaard

This comment has been minimized.

Show comment
Hide comment
@jon-a-nygaard

jon-a-nygaard Jul 20, 2016

Collaborator

Reopening the issue, since the fix has not yet been merged into the master branch. 😄

Collaborator

jon-a-nygaard commented Jul 20, 2016

Reopening the issue, since the fix has not yet been merged into the master branch. 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment