Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Netty Connector doesn't support Followredirects. #5045

Closed
sumitaneja opened this issue Apr 21, 2022 · 5 comments
Closed

Netty Connector doesn't support Followredirects. #5045

sumitaneja opened this issue Apr 21, 2022 · 5 comments
Assignees
@jbescos

Comments

@sumitaneja
Copy link

Follow redirect(jersey.config.client.followRedirects) is true but netty connector still doesn't follow redirect and provides 307 to client.
@jbescos
Copy link
Member

jbescos commented Apr 22, 2022

Hi @sumitaneja I have created an issue in Netty to support follow redirects.

We will see whether they can implement this, or we have to implement it in Jersey.

@jansupol
Copy link
Contributor

hm... I think Netty supports follow-redirect, at least HelidonConnector based on Netty already implements follow-redirect.

@jansupol
Copy link
Contributor

it might be possible that Helidon WebClient supports follow-redirect on its own, without Netty, though...

@jbescos
Copy link
Member

jbescos commented Apr 22, 2022 •
edited
Loading

it might be possible that Helidon WebClient supports follow-redirect on its own, without Netty, though...

I think Helidon manages that on its own in io.helidon.webclient.RedirectInterceptor

@jbescos jbescos self-assigned this Apr 25, 2022
@jbescos jbescos mentioned this issue Apr 25, 2022
Merged
@jansupol
Copy link
Contributor

Closed by #5048

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@jbescos jbescos

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jbescos @jansupol @sumitaneja