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.

Sign up for GitHub

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

Jump to bottom

Connector cannot resolve 301/302 redirects #13

Open
tomkatsu opened this issue Sep 2, 2020 · 2 comments
Open

Connector cannot resolve 301/302 redirects #13

tomkatsu opened this issue Sep 2, 2020 · 2 comments
Labels
bug Something isn't working

Comments

@tomkatsu
Copy link
Contributor

tomkatsu commented Sep 2, 2020

Connector outputs the following when a 301 is encountered:
663 [main] ERROR com.lucidworks.fusion.connector.service.DrupalHttpClient - Error on the request getting the content. Status-Code: 301

Expected behavior is to follow redirects to their destination.
@tomkatsu
Copy link
Contributor Author

tomkatsu commented Sep 2, 2020

These should specifically include redirects from http -> https. We've got JSON being output from Drupal that have http links with the webserver 301ing it to https.

@tomkatsu tomkatsu added the bug Something isn't working label Sep 3, 2020
@cristinalola
Copy link
Contributor

I've added a fix for this in bugfix/LCW-154 branch. I've added the 3xx redirects manually, still using HttpUrlConnection from Java Core. I've tried using Apache HttpClient but at some point the login request would stop working, so I continued using Java Core for requests.

https://github.com/lucidworks/drupal-connector/tree/bugfix/LCW-154

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tomkatsu @cristinalola