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

KEYCLOAK-7677 KEYCLOAK-7723 fix version collision of httpclient #5795

Conversation

vramik
Copy link
Contributor

@vramik vramik commented Dec 7, 2018

No description provided.

@vramik vramik changed the title KEYCLOAK-7677 KEYCLOAK-7723 fix version collision of httpclient WIP KEYCLOAK-7677 KEYCLOAK-7723 fix version collision of httpclient Dec 7, 2018
@stianst stianst requested a review from pdrozd December 11, 2018 13:25
@vramik vramik force-pushed the KEYCLOAK-7677-fix-httpclient-version-colision branch 2 times, most recently from 2cbbd1e to b1a8c12 Compare December 18, 2018 09:43
@vramik vramik force-pushed the KEYCLOAK-7677-fix-httpclient-version-colision branch from b1a8c12 to f901680 Compare January 4, 2019 10:19
@pedroigor
Copy link
Contributor

@vramik, could you please incorporate this pedroigor@ef8cf70 to your PR. The ImpersonationTest should run fine now.

If you want I can close your PR and send another preserving your commits, if you don't have time to work on that.

@vramik vramik force-pushed the KEYCLOAK-7677-fix-httpclient-version-colision branch 2 times, most recently from d86ef7a to 0b44734 Compare January 10, 2019 10:45
hmlnarik
hmlnarik previously approved these changes Jan 10, 2019
Copy link
Contributor

@hmlnarik hmlnarik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One nit left, feel free to let it be if no other issue is raised.

@hmlnarik hmlnarik dismissed their stale review January 10, 2019 11:47

The travis failures seem related, could you please check?

@vramik vramik force-pushed the KEYCLOAK-7677-fix-httpclient-version-colision branch from 0b44734 to eea938d Compare January 10, 2019 13:23
Co-authored-by: Pedro Igor <psilva@redhat.com>
@vramik vramik force-pushed the KEYCLOAK-7677-fix-httpclient-version-colision branch from eea938d to 10486f3 Compare January 10, 2019 13:48
@vramik vramik changed the title WIP KEYCLOAK-7677 KEYCLOAK-7723 fix version collision of httpclient KEYCLOAK-7677 KEYCLOAK-7723 fix version collision of httpclient Jan 10, 2019
@pedroigor pedroigor merged commit c4a46a5 into keycloak:master Jan 10, 2019
@vramik vramik deleted the KEYCLOAK-7677-fix-httpclient-version-colision branch January 11, 2019 08:11
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

Successfully merging this pull request may close these issues.

4 participants