Uploaded image for project: 'Kernel'
  1. Kernel
  2. KNL-213

BaseResourceEdit return non url-encoded urls

    Details

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

      Description

      BaseResourceEdit (and other classes in BaseContentService.java) returns urls that are not url encoded, thus breaking on filenames with characters that have to be encoded.

      For us this breaks uploaded files in atleast syllabus and test&quizzes

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            buckett Matthew Buckett added a comment -

            A merge to 1.0.x is need to fix what issue in 2.6?
            As Stephen mentioned I think this is a risky issue to merge back into 2.6/K1 due to the risk of double escaping.

            Show
            buckett Matthew Buckett added a comment - A merge to 1.0.x is need to fix what issue in 2.6? As Stephen mentioned I think this is a risky issue to merge back into 2.6/K1 due to the risk of double escaping.
            Hide
            arwhyte Anthony Whyte added a comment -

            Bumped to 1.0.14. However, a more formal decision should be rendered regarding whether or not to include this fix in 1.0.x given the range of 2.6 tools that require fixing in order to implement the fix properly.

            Show
            arwhyte Anthony Whyte added a comment - Bumped to 1.0.14. However, a more formal decision should be rendered regarding whether or not to include this fix in 1.0.x given the range of 2.6 tools that require fixing in order to implement the fix properly.
            Hide
            dhorwitz David Horwitz added a comment -

            due to the impact of this change it won't be fixed at this point in the 1.0 series

            Show
            dhorwitz David Horwitz added a comment - due to the impact of this change it won't be fixed at this point in the 1.0 series
            Hide
            nicola@nyu.edu Nicola Monat-Jacobs added a comment -

            Since this was never back-ported to the 1.0.x branch, can we remove the '1.0.14' fix version?

            Show
            nicola@nyu.edu Nicola Monat-Jacobs added a comment - Since this was never back-ported to the 1.0.x branch, can we remove the '1.0.14' fix version?
            Hide
            smarquard Stephen Marquard added a comment -

            Changed Fix Version as per discussion above.

            Show
            smarquard Stephen Marquard added a comment - Changed Fix Version as per discussion above.

              People

              • Assignee:
                jimeng Jim Eng
                Reporter:
                kllin Klas Lindfors
              • Votes:
                0 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development

                    Git Source Code