Uploaded image for project: 'Hippo Site Toolkit 2'
  1. Hippo Site Toolkit 2
  2. HSTTWO-4493

Log warning instead of ignoring unsupported cdn host configuration

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Top
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.2.11, 5.6.1, 13.0.0
    • Component/s: None
    • Labels:
      None
    • Similar issues:
    • Flagged:
      Flagged
    • Processed by team:
      Turing
    • Sprint:
      Turing Sprint 195, Turing Sprint 196, Turing Sprint 197

      Description

      The current use case is that we need to specify https:// explicitly since the url is also used in generated json files which are used by a third party app. The current implementation logs and error and ignores the setting if the url does not start with '//' or ends with '/'. The attached patch changes this behavior so that a warning is logged but the value is used hence giving back the control (to break) back to developers.

      There are no functional changes except for relaxing the strictness on the configuration value.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              bvdschans Bart van der Schans (Inactive)
              Owner:
              Hippo Helpdesk
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: