Skip to content

CFE-4188: Removed requests python dependency in cf-bottom - #119

Merged
victormlg merged 1 commit into
cfengine:masterfrom
victormlg:remove-requests-dep
Jun 3, 2026
Merged

victormlg merged 1 commit into
cfengine:masterfrom
victormlg:remove-requests-dep

Conversation

@victormlg

Copy link
Copy Markdown
Contributor

No description provided.

@victormlg
victormlg force-pushed the remove-requests-dep branch from d6816fa to c5245fb Compare March 31, 2026 11:37
@cf-bottom

Copy link
Copy Markdown
Collaborator

Thanks for submitting a PR! Maybe @larsewi can review this?

@victormlg
victormlg force-pushed the remove-requests-dep branch 2 times, most recently from 3aaa07c to 2b588d3 Compare June 2, 2026 12:25
@victormlg
victormlg requested a review from larsewi June 2, 2026 12:27
Comment thread tom/network.py Outdated
Ticket: CFE-4188
Signed-off-by: Victor Moene <victor.moene@northern.tech>
@victormlg
victormlg force-pushed the remove-requests-dep branch from 2b588d3 to db10aad Compare June 3, 2026 10:50
@victormlg
victormlg merged commit 9e23ea2 into cfengine:master Jun 3, 2026
6 checks passed
olehermanse pushed a commit that referenced this pull request Oct 16, 2026
Introduced in #119, which replaced requests with a Response class
that has no ok attribute. Check status_code instead.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
olehermanse pushed a commit that referenced this pull request Oct 16, 2026
Introduced in #119. Callers expecting the error JSON crashed on an
empty body.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants