Uploaded image for project: 'Magnolia UI'
  1. Magnolia UI
  2. MGNLUI-7379

Global validator SafeHtmlValidatorDefinition should accept HTML comment block

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Neutral
    • 6.2.26
    • 6.2.21
    • None
    • None

    Description

      Steps to reproduce

      1.  Enable source mode for any Rich Text Field
      2.  Put an HTML comment into "source" mode with some valid text and tags
      3. Save the component

      .. Logs, screenshots, gifs...

      Expected results

      • Content save without any error

      .. Justify non-trivial expectations with a link to a doc or a relevant discussion.

      Actual results

      • Users cannot save content, "Malicious HTML code found" validation error occurred.

      Workaround

      • Please ask users/customers to temporarily remove the comment(s) from your HTML by using any HTML editor / online editor tool for that.

      Development notes

      • N/A

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            caused by

            Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. MGNLUI-7285 richTextField HTML validation

            • Neutral -
            • Selected
            is cloned by

            Improvement - An improvement or enhancement to an existing feature or task. MGNLUI-7570 Magnolia 6.3 - Global validator SafeHtmlValidatorDefinition does not accept HTML comment block

            • Neutral -
            • Closed

            Activity

              People

                sang.ngo Sang Ngo Huu
                viet.nguyen Viet Nguyen
                Nucleus
                Votes:
                1 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Task DoD

                    Time Tracking

                      Estimated:
                      Original Estimate - Not Specified
                      Not Specified
                      Remaining:
                      Remaining Estimate - Not Specified
                      Not Specified
                      Logged:
                      Time Spent - 2d
                      2d