Kuali Rice Development
  1. Kuali Rice Development
  2. KULRICE-14257

Invalid Unicode control characters cause failures in serializing document fields for SOAP requests

    Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.6
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Similar issues:
      KULRICE-4496Speed Test: Soap vs java serialization for our remoted services
      KULRICE-8180Need a way to control what properties are serialized in maintenance documents.
      KULRICE-11598Throw exception if component IDs contain invalid characters
      KULRICE-12033Add annotation to enable serialization opt-in / opt-out for individual maintenance doc fields
      KULRICE-4700Fully convert java serialized services to SOAP services
      KULRICE-13128AFT Failure LabsView ContainerAfts Freemarker Exception Invalid property 'backdoorId'
      KULRICE-6836Add feature to remove special characters for text area fields
      KULRICE-8908To make document explanation field controllable of editing permission
      KULRICE-5867Make it so that document does not crash if an invalid network ID is passed in for an eDocLite field.
      KULRICE-11597Generated lookup/inquiry ids contain invalid characters
    • Rice Module:
      KEW
    • Sprint:
      Rice Sprint 2015-05-13
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Code Review Status:
      Not Required
    • Contributing Institution:
      Indiana Univ
    • Include in Release Notes?:
      Yes

      Description

      Documents (or any entity transferred over SOAP) which contain fields with user-entered Strings (generally the title) can contain Unicode control characters that are invalid under the XML 1.0 standard. Serializing and deserializing such a document will cause a SOAP exception.

        Activity

        Hide
        Francis Fernandez added a comment -

        Created Pull Request #178

        Show
        Francis Fernandez added a comment - Created Pull Request #178
        Hide
        Claus Niesen added a comment -

        Shannon, can you review the contribution. Specifically if the xalan dependency is needed. I think the dependency should probably have an explicit version number set. BTW, the pull request test job has been started already. Thanks.

        Show
        Claus Niesen added a comment - Shannon, can you review the contribution. Specifically if the xalan dependency is needed. I think the dependency should probably have an explicit version number set. BTW, the pull request test job has been started already. Thanks.
        Hide
        Shannon Hess added a comment -

        Thanks for creating the pull request, I'll take a look.

        Show
        Shannon Hess added a comment - Thanks for creating the pull request, I'll take a look.
        Hide
        Shannon Hess added a comment -

        This looks good. I brought in the patch and ran the test. The xalan dependency is needed to bring in org.apache.xml.utils.XMLChar. There is no version but that is typical for dependencies in rice-middleware/kew/api/pom.xml. The version for xalan is set in the overall rice pom.xml ( <xalan.version>2.7.2</xalan.version> ).

        Show
        Shannon Hess added a comment - This looks good. I brought in the patch and ran the test. The xalan dependency is needed to bring in org.apache.xml.utils.XMLChar. There is no version but that is typical for dependencies in rice-middleware/kew/api/pom.xml. The version for xalan is set in the overall rice pom.xml ( <xalan.version>2.7.2</xalan.version> ).
        Hide
        Shannon Hess added a comment -

        The pull request has been merged

        Show
        Shannon Hess added a comment - The pull request has been merged
        Hide
        Claus Niesen added a comment -

        Thanks Francis and Shannon.

        Show
        Claus Niesen added a comment - Thanks Francis and Shannon.

          People

          • Assignee:
            Shannon Hess
            Reporter:
            Francis Fernandez
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Agile

                Structure Helper Panel