Weather Portlet
  1. Weather Portlet
  2. WPT-47

Yahoo provider causes mixed-content warnings

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      When using the Yahoo provider, the images are non-secure URLs, which causes mixed content warnings in the browser. Can anything be done within the portlet to avoid this?

        Activity

        Hide
        Holly Coast added a comment -

        The World Weather Online provider does not cause mixed content warnings. However, they have a limit on the number of connections per hour, and with our high usage volume I'm not sure that will work for us in our production environment.

        Show
        Holly Coast added a comment - The World Weather Online provider does not cause mixed content warnings. However, they have a limit on the number of connections per hour, and with our high usage volume I'm not sure that will work for us in our production environment.
        Hide
        Aaron Grant added a comment -

        I was researching this as we are noticing this issue also. It appears this URL can be used instead:

        http://l.yimg.com/a/i/us/we/52/33.gif -> https://s.yimg.com/zz/combo?/a/i/us/we/52/33.gif
        https://developer.yahoo.com/forums/#/discussion/6313/weather-api-https-images

        I'll see if I can get a patch for this done and do a pull request.

        Show
        Aaron Grant added a comment - I was researching this as we are noticing this issue also. It appears this URL can be used instead: http://l.yimg.com/a/i/us/we/52/33.gif -> https://s.yimg.com/zz/combo?/a/i/us/we/52/33.gif https://developer.yahoo.com/forums/#/discussion/6313/weather-api-https-images I'll see if I can get a patch for this done and do a pull request.
        Hide
        Aaron Grant added a comment -
        Show
        Aaron Grant added a comment - Pull request accepted here: https://github.com/Jasig/WeatherPortlet/pull/16

          People

          • Assignee:
            Aaron Grant
            Reporter:
            Holly Coast
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: