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

          Attachments

            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:

                    Git Source Code