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

apparently circular redirect makes p:http-request crash #235

Open
gimsieke opened this issue Feb 1, 2016 · 0 comments
Open

apparently circular redirect makes p:http-request crash #235

gimsieke opened this issue Feb 1, 2016 · 0 comments

Comments

@gimsieke
Copy link
Contributor

gimsieke commented Feb 1, 2016

When checking URL citations in journal XML with p:http-request, we recently stumbled across the URL http://www.journalofosteopathicmedicine.com/ that makes p:http-request (and the whole pipeline and Calabash around it) crash.

This gist contains some information on how to reproduce it.

The site gives cookies and redirects with 302. I think the HTTP client uses these cookies when following the redirect, but then it will always be redirected again to the same address, as @lwittmar found out:

Caused by: org.apache.http.client.CircularRedirectException: 
Circular redirect to 'http://www.journalofosteopathicmedicine.com/action
/consumeSharedSessionAction?JSESSIONID=aaadi4L48DSXp-opBjfkv&
MAID=3hkdvCRbSm1D9doWM4MieA%3D%3D&
SERVER=WZ6myaEXBLFhx%2B6Ws3Nrug%3D%3D&
ORIGIN=540506413&RD=RD'

If you enter the URL in a web browser, you’ll see the journal’s landing page.

If, for some reason, the HTTP client is unable to finally get a 200, Calabash should at least gracefully catch the exception, methinks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant