Uploaded image for project: 'SwitchYard'
  1. SwitchYard
  2. SWITCHYARD-425

BeanSwitchYardScanner is not able to scan a project if the path contains a plus sign

        Details

        • Type: Bug
        • Status: Closed (View Workflow)
        • Priority: Minor
        • Resolution: Done
        • Affects Version/s: 0.2
        • Fix Version/s: 0.3
        • Component/s: component-bean
        • Labels:
          None
        • Environment:
        • Steps to Reproduce:
          Hide

          I am going to add a reference a to a topic branch with a unit test that demonstrate this.

          Show
          I am going to add a reference a to a topic branch with a unit test that demonstrate this.
        • Workaround Description:
          Hide

          The problem is that the + signs are replaced by spaces when this line of code is executed in ClasspathScanner::toClassPathFile(...) (in core/common/src/main/java/org/switchyard/common/type/classpath/)

          urlPath = URLDecoder.decode(urlPath, "UTF-8");

          Show
          The problem is that the + signs are replaced by spaces when this line of code is executed in ClasspathScanner::toClassPathFile(...) (in core/common/src/main/java/org/switchyard/common/type/classpath/) urlPath = URLDecoder.decode(urlPath, "UTF-8");
        • Git Pull Request:

          Description

          When a project is located inside a path that contains a + sign, the BeanSwitchYardScanner fails.

          This can be as a quite unusual matter but in fact in my computer all the tests are run in a temp directory automatically generated which contains some + signs.

            Gliffy Diagrams

              Issue Links

              blocks

              Task - A task that needs to be done. SWITCHYARD-391 Add tests for Forge plugins

              • Major - A request that should be considered seriously but is not a show stopper.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                antollinim Mario Antollini added a comment -

                This is the unit test that demonstrates the existence of the issue:

                https://github.com/antollinim/components/commit/92f18bd67ee126087ae8543dc091da84ec793b03

                Show
                antollinim Mario Antollini added a comment - This is the unit test that demonstrates the existence of the issue: https://github.com/antollinim/components/commit/92f18bd67ee126087ae8543dc091da84ec793b03
                Hide
                Permalink
                kcbabo Keith Babo added a comment -

                pushed

                Show
                kcbabo Keith Babo added a comment - pushed

                  People

                  • Assignee:
                    tfennelly Tom Fennelly
                    Reporter:
                    antollinim Mario Antollini
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    1 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved:

                      Development