ICEpush
  1. ICEpush
  2. PUSH-9

Extra Newlines being appended to create-push-id response.

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: None
          • Fix Version/s: 2.0-Alpha2
          • Component/s: None
          • Labels:
            None
          • Environment:
            all

            Description

            There are two extra newlines being appended to the end of the create-push-id ajax response. This is leading to an incorrect pushId being returned via the JS push API and the subsequent call to addGroupMembership is registering the wrong id to a group - thus no notifications are recieved.

            Proposed solution:

            1.) trim() the pushId before returning it to the user

            2.) BrowserBoundServlet.java (line 28) . Delete this line.

              Activity

              Ascending order - Click to sort in descending order
              Patrick Wilson created issue -
              Hide
              Permalink
              Patrick Wilson added a comment -

              Assigning to Mircea for analysis.

              Show
              Patrick Wilson added a comment - Assigning to Mircea for analysis.
              Patrick Wilson made changes -
              Field Original Value New Value
              Assignee Mircea Toma [ mircea.toma ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #20107 Mon Jan 04 14:08:17 MST 2010 mircea.toma PUSH-9 Stop sending additional '\n' characters in response.
              Files Changed
              Commit graph MODIFY /scratchpads/snowplow/core/src/main/java/org/icepush/servlet/BrowserBoundServlet.java
              Hide
              Permalink
              Mircea Toma added a comment -

              Stop sending additional '\n' characters in response.

              Show
              Mircea Toma added a comment - Stop sending additional '\n' characters in response.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 1.0-Alpha2 [ 10223 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Mircea Toma
                  Reporter:
                  Patrick Wilson
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: