Skip to content

Update http_request_handler.go#1833

Open
zhao85 wants to merge 3 commits into
cortezaproject:2023.9.xfrom
zhao85:2023.9.x
Open

Update http_request_handler.go#1833
zhao85 wants to merge 3 commits into
cortezaproject:2023.9.xfrom
zhao85:2023.9.x

Conversation

@zhao85
Copy link
Copy Markdown

@zhao85 zhao85 commented Jun 15, 2024

fix http context exceptional cancel problem

The following changes are implemented

fix http context exceptional cancel problem

Changes in the user interface:

TODO: Add screenshots, recordings or remove this section

Checklist when submitting a final (!draft) PR

  • Commits are tidied up, squashed if needed and follow guidelines in CONTRIBUTING.md
  • Code builds
  • All existing tests pass
  • All new critical code is covered by tests
  • PR is linked to the relevant issue(s)
  • Rebased with the target branch

fix http context exceptional cancel problem

Signed-off-by: zhaoyi233 <11971297+zhao85@users.noreply.github.com>
@Fajfa Fajfa changed the base branch from 2023.9.x to 2024.9.x September 30, 2024 11:45
@Fajfa Fajfa changed the base branch from 2024.9.x to 2023.9.x September 30, 2024 11:45
@Fajfa
Copy link
Copy Markdown
Member

Fajfa commented Sep 30, 2024

@zhao85 Hey im cleaning PR's, can you describe what this is please?

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 5, 2025

Stale pull request message

@jackmaninov
Copy link
Copy Markdown

I am seeing the same issue (described in #1830), I will try pulling this PR and see if it corrects things for me.

@jackmaninov
Copy link
Copy Markdown

No, this PR doesn't seem to help with my problem (a workflow that does an HTTP request erroring with Context Cancelled within about 2 seconds of starting).

@Fajfa Fajfa force-pushed the 2023.9.x branch 2 times, most recently from 3f44fa3 to 96ef3c5 Compare July 11, 2025 14:36
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 1, 2025

Stale pull request message

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants