Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parameter parsing problem in 1.1.0-SNAPSHOT #691

Closed
jeremy303 opened this issue Oct 15, 2012 · 4 comments
Closed

Parameter parsing problem in 1.1.0-SNAPSHOT #691

jeremy303 opened this issue Oct 15, 2012 · 4 comments
Labels

Comments

@jeremy303
Copy link

Just just changed my Atmosphere version from 1.1-SNAPSHOT to 1.1.0-SNAPSHOT and have encountered the following issue.

The client GETs the following URL:
/locations?page_start=UHhE2uSOGK9y-gta&page_limit=10&page_hint=next

In Jersey, the first query parameter in my method receives a concatenation of all the query params without the '?' delimiters, like so:
UHhFQuSOho-6eCHBpage_limit=10page_hint=next

If I roll the Atmosphere version back to 1.1-SNAPSHOT, all is well.

@jfarcand
Copy link
Member

Salut, I cannot reproduce the issue. Can you share a test case?

@jeremy303
Copy link
Author

Sure thing. Thanks.

From: Jeanfrancois Arcand [mailto:notifications@github.com]
Sent: Friday, October 19, 2012 2:27 PM
To: Atmosphere/atmosphere
Cc: Jeremy Johnson
Subject: Re: [atmosphere] Parameter parsing problem in 1.1.0-SNAPSHOT (#691)

Salut, I cannot reproduce the issue. Can you share a test case?


Reply to this email directly or view it on GitHubhttps://github.com//issues/691#issuecomment-9611566.

@jeremy303
Copy link
Author

Ok, finally made this test case:
https://gist.github.com/08ba24e77913537a2e5c

Sorry it's taken forever. I guess I shouldn't expect a quick solution in return, eh? ;-)

Here are the atmosphere / jersey / jetty versions used:

    <atmosphere-version>1.1.0-SNAPSHOT</atmosphere-version>
    <jersey-version>1.13</jersey-version>
    <jetty-version>8.1.5.v20120716</jetty-version>

As always, thanks!

@jfarcand
Copy link
Member

jfarcand commented Dec 4, 2012

LOL! I will cut 1.1.beta2 soon and will make sure it gets added. Thanks!!!!!

jfarcand added a commit that referenced this issue Dec 13, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants