Skip to content

fix: match token secrets that keep the provider name in the token name (#1090) - #1091

Merged
vinokurig merged 1 commit into
7.123.xfrom
CRW-13991_7.123.x
Oct 23, 2026
Merged

vinokurig merged 1 commit into
7.123.xfrom
CRW-13991_7.123.x

Conversation

@vinokurig

Copy link
Copy Markdown
Contributor
  • fix: match token secrets that keep the provider name in the token name

isSecretMatchesSearchCriteria read the OAuth provider name from the che.eclipse.org/scm-provider-name annotation only. Secrets stored before that annotation was introduced keep the provider name in che.eclipse.org/scm-personal-access-token-name, so a lookup by provider name stopped matching them. Fall back to the token name when the provider name annotation is absent or empty, the same way getScmProviderName already does.

  • fixup! fix: match token secrets that keep the provider name in the token name

What does this PR do?

Screenshot/screencast of this PR

What issues does this PR fix or reference?

How to test this PR?

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.

#1090)

* fix: match token secrets that keep the provider name in the token name

isSecretMatchesSearchCriteria read the OAuth provider name from the
che.eclipse.org/scm-provider-name annotation only. Secrets stored before
that annotation was introduced keep the provider name in
che.eclipse.org/scm-personal-access-token-name, so a lookup by provider
name stopped matching them. Fall back to the token name when the provider
name annotation is absent or empty, the same way getScmProviderName
already does.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

* fixup! fix: match token secrets that keep the provider name in the token name

---------

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
@openshift-ci

openshift-ci Bot commented Oct 23, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 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

@coderabbitai

coderabbitai Bot commented Oct 23, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: e79e9a44-ea50-4bdf-b03d-8a1e5962f94c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

Docker image build succeeded: quay.io/eclipse/che-server:pr-1091

kubectl patch command
kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/cheServer/deployment", "value": {containers: [{image: "quay.io/eclipse/che-server:pr-1091", name: che}]}}]"

@vinokurig
vinokurig merged commit 2b2f60b into 7.123.x Oct 23, 2026
7 checks passed
@vinokurig
vinokurig deleted the CRW-13991_7.123.x branch October 23, 2026 15:13
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.

1 participant