ICEpush
  1. ICEpush
  2. PUSH-48

appendToURI() does not check for existing parameter in uri

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.0-Alpha2
          • Fix Version/s: 2.0-Alpha3
          • Component/s: Java Client
          • Labels:
            None
          • Environment:
            n/a

            Description

            in http.js:

            appendToURI(parameters,uri) doesn't check if the uri already has parameters and will create a uri that includes multiple '?' characters, breaking the parameters

              Issue Links

              blocks

              Task - A task that needs to be done. PUSH-46 ICEchat JSP version

              • Major - Major loss of function.
              • Closed

                Activity

                Descending order - Click to sort in ascending order
                Ken Fyten made changes -
                Status Resolved [ 5 ] Closed [ 6 ]
                Mircea Toma made changes -
                Fix Version/s 1.0-Alpha3 [ 10224 ]
                Affects Version/s 1.0-Alpha2 [ 10223 ]
                Affects Version/s 1.0-Alpha3 [ 10224 ]
                Mircea Toma made changes -
                Status Open [ 1 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Hide
                Permalink
                Mircea Toma added a comment -

                Check if the provided URI already includes a query before constructing the complete query.

                Show
                Mircea Toma added a comment - Check if the provided URI already includes a query before constructing the complete query.
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #20845 Tue Mar 02 16:22:32 MST 2010 mircea.toma PUSH-48 Check if the provided URI already includes a query before constructing the complete query.
                Files Changed
                Commit graph MODIFY /projects/bridge-support/src/main/javascript/http.js
                Ken Fyten made changes -
                Assignee Mircea Toma [ mircea.toma ]
                Philip Breau made changes -
                Field Original Value New Value
                Link This issue blocks PUSH-46 [ PUSH-46 ]
                Hide
                Permalink
                Philip Breau added a comment -

                recommended patch

                Index: /home/philip/workspace/ICEpush/core/src/main/javascript/lib/http.js
                ===================================================================
                — /home/philip/workspace/ICEpush/core/src/main/javascript/lib/http.js (revision 20781)
                +++ /home/philip/workspace/ICEpush/core/src/main/javascript/lib/http.js (working copy)
                @@ -73,7 +73,7 @@

                method(appendToURI, function(self, uri) {
                if (not(isEmpty(parameters)))

                { - return uri + '?' + asURIEncodedString(self); + return uri + ( uri.indexOf('?') == -1 ? '?' : '&' ) + asURIEncodedString(self); }

                else

                { return uri; }
                Show
                Philip Breau added a comment - recommended patch Index: /home/philip/workspace/ICEpush/core/src/main/javascript/lib/http.js =================================================================== — /home/philip/workspace/ICEpush/core/src/main/javascript/lib/http.js (revision 20781) +++ /home/philip/workspace/ICEpush/core/src/main/javascript/lib/http.js (working copy) @@ -73,7 +73,7 @@ method(appendToURI, function(self, uri) { if (not(isEmpty(parameters))) { - return uri + '?' + asURIEncodedString(self); + return uri + ( uri.indexOf('?') == -1 ? '?' : '&' ) + asURIEncodedString(self); } else { return uri; }
                Philip Breau created issue -

                  People

                  • Assignee:
                    Mircea Toma
                    Reporter:
                    Philip Breau
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    0 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: