Skip to content

[7.117] Replace axios with fetch + https fallback in che-api/github-service#145

Draft
RomanNikitenko wants to merge 4 commits into
7.117.xfrom
test-axios
Draft

[7.117] Replace axios with fetch + https fallback in che-api/github-service#145
RomanNikitenko wants to merge 4 commits into
7.117.xfrom
test-axios

Conversation

@RomanNikitenko

@RomanNikitenko RomanNikitenko commented May 15, 2026

Copy link
Copy Markdown
Owner

What does this PR do?

What issues does this PR fix?

How to test this PR?

  1. Start mitmproxy on the host:

    mitmdump --listen-port 8080

2.1 Set your token in the command below
2.2 Run the che-code container with proxy settings and the mitmproxy CA:

podman run --rm -it -p 3100:3100 \
-e CODE_HOST=0.0.0.0 \
-e HTTPS_PROXY=http://host.containers.internal:8080 \
-e HTTP_PROXY=http://host.containers.internal:8080 \
-e NODE_EXTRA_CA_CERTS=/public-certs/mitmproxy-ca.crt \
-e GITHUB_TEST_TOKEN=token \
-v ~/.mitmproxy/mitmproxy-ca-cert.pem:/public-certs/mitmproxy-ca.crt:ro \
quay.io/rnikitenko/che-code:pr-145-arm64  
  1. Open the editor in a browser (incognito window) at http://localhost:3100 (see URL in the command output).

  2. F1 => Test GitHub Proxy (axios) => see notification and check logs(which mechanism is used? the main? the fallback?) in the che-api output

image
  1. Run the editor on openshift => Trigger any flow that calls che-api's GithubService.getUser() (e.g. the Device Authentication flow after completing GitHub OAuth).

Does this PR contain changes that override default upstream Code-OSS behavior?

  • the PR contains changes in the code folder (you can skip it if your changes are placed in a che extension )
  • the corresponding items were added to the CHANGELOG.md file
  • rules for automatic git rebase were added to the .rebase folder

@github-actions

github-actions Bot commented May 15, 2026

Copy link
Copy Markdown

Click here to review and test in web IDE: Contribute

@github-actions

Copy link
Copy Markdown

Pull Request images published ✨

Editor amd64: quay.io/rnikitenko/che-code:pr-145-amd64

1 similar comment
@github-actions

Copy link
Copy Markdown

Pull Request images published ✨

Editor amd64: quay.io/rnikitenko/che-code:pr-145-amd64

@github-actions

Copy link
Copy Markdown

Pull Request images published ✨

Editor amd64: quay.io/rnikitenko/che-code:pr-145-amd64

@github-actions

Copy link
Copy Markdown

Pull Request images published ✨

Editor amd64: quay.io/rnikitenko/che-code:pr-145-amd64
Editor arm64: quay.io/rnikitenko/che-code:pr-145-arm64

1 similar comment
@github-actions

Copy link
Copy Markdown

Pull Request images published ✨

Editor amd64: quay.io/rnikitenko/che-code:pr-145-amd64
Editor arm64: quay.io/rnikitenko/che-code:pr-145-arm64

Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
Assisted-by: Cursor AI
Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
Assisted-by: Cursor AI
@RomanNikitenko RomanNikitenko changed the title [7.116] Test Axios problem [7.117] Test Axios problem May 18, 2026
@RomanNikitenko RomanNikitenko changed the title [7.117] Test Axios problem [7.117] Replace axios with fetch + https fallback in che-api/github-service May 18, 2026
@RomanNikitenko RomanNikitenko changed the base branch from 7.116.x to 7.117.x May 18, 2026 10:20
Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
Assisted-by: Cursor AI
@github-actions

Copy link
Copy Markdown

Pull Request images published ✨

Editor amd64: quay.io/rnikitenko/che-code:pr-145-amd64
Editor arm64: quay.io/rnikitenko/che-code:pr-145-arm64

1 similar comment
@github-actions

Copy link
Copy Markdown

Pull Request images published ✨

Editor amd64: quay.io/rnikitenko/che-code:pr-145-amd64
Editor arm64: quay.io/rnikitenko/che-code:pr-145-arm64

…hub-service

Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
Assisted-by: Cursor AI
@github-actions

Copy link
Copy Markdown

Pull Request images published ✨

Editor amd64: quay.io/rnikitenko/che-code:pr-145-amd64
Editor arm64: quay.io/rnikitenko/che-code:pr-145-arm64

1 similar comment
@github-actions

Copy link
Copy Markdown

Pull Request images published ✨

Editor amd64: quay.io/rnikitenko/che-code:pr-145-amd64
Editor arm64: quay.io/rnikitenko/che-code:pr-145-arm64

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