Skip to content

Conversation

@vinokurig
Copy link
Contributor

@vinokurig vinokurig commented Jan 22, 2026

What does this PR do?

Add missing dependency for the google-http-client library

Screenshot/screencast of this PR

What issues does this PR fix or reference?

fixes eclipse-che/che#23721

How to test this PR?

Apply the pull request image: quay.io/eclipse/che-server:pr-925,
see eclipse-che/che#23721

PR Checklist

As the author of this Pull Request I made sure that:

Release Notes

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Signed-off-by: Ihor Vinokur <ivinokur@redhat.com>
@vinokurig
Copy link
Contributor Author

/retest

@vinokurig vinokurig changed the title fix: Add missing dependency for google-http-client library fix: Add missing dependency for the google-http-client library Jan 22, 2026
Copy link
Contributor

@svor svor left a comment

Choose a reason for hiding this comment

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

oAuth authorization flow works as expected

Screenshot 2026-01-22 at 12 29 55

@openshift-ci openshift-ci bot added the lgtm label Jan 22, 2026
@svor svor requested a review from artaleks9 January 22, 2026 11:29
@artaleks9
Copy link
Contributor

Verified on Eclipse Che deployed on OCP v20 with quay.io/eclipse/che-server:pr-925

The functionality works properly

@openshift-ci
Copy link

openshift-ci bot commented Jan 22, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: artaleks9, svor, vinokurig

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@svor svor merged commit fffca7d into main Jan 22, 2026
29 checks passed
@svor svor deleted the che-23721 branch January 22, 2026 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

An error appears on oAuth authorization

4 participants