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

JEEFormAuthenticator does not try to append cookies to a fresh connection #257

Closed

berezovskyi opened this issue Feb 20, 2022 · 1 comment · Fixed by #258

Labels

Component: Client Priority: Medium Type: Enhancement

Milestone

Contributor

berezovskyi commented Feb 20, 2022 •

edited

Loading

See https://github.com/eclipse/lyo/blob/7a83a881facdf94bbed032d1f509b2085a39e6b3/client/oslc-client/src/main/java/org/eclipse/lyo/client/JEEFormAuthenticator.java#L102-L107, the filter does nothing unless it's processing a 401 response.

This is likely what causes a performance regression causing excessive number of auth requests with Jazz apps, as reported on the forum.

berezovskyi added Type: Enhancement Priority: Medium Component: Client labels

berezovskyi added this to the 5.0 milestone

Contributor Author

berezovskyi commented Feb 20, 2022

Since AHC4.3, there is a possibility of doing more advanced connection management and pooling in AHC, but we should consider this in connection to #246 to avoid doing work that will become obsolete soon.

berezovskyi mentioned this issue

Try to reuse credentials in JEEFormAuthenticator #258

Merged

3 tasks

berezovskyi closed this as completed in #258

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment