fix: use fetch for DELETE+body to fix file_delete (#649) #20

Merged

jmiller merged 1 commits from fix/649-file-delete-body into main 2026-06-20 20:01:47 +00:00

Conversation 0 Commits 1 Files Changed 1

+15 -1

jmiller commented 2026-06-20 19:58:46 +00:00

Owner

Copy Link

Copy Source

Summary\n\nFixes MokoGitea-Fork#649 (file_delete bug)\n\nnode:https drops the request body on DELETE requests through some proxy/TLS configurations, causing Gitea to see an empty body and return "file already exists" (routes to CREATE handler).\n\nSwitches to built-in fetch() for DELETE requests with a body, which handles this correctly.\n\n## Changes\n\n- src/client.ts — delete() method uses fetch() when body is present

## Summary\n\nFixes MokoGitea-Fork#649 (file_delete bug)\n\nnode:https drops the request body on DELETE requests through some proxy/TLS configurations, causing Gitea to see an empty body and return \"file already exists\" (routes to CREATE handler).\n\nSwitches to built-in fetch() for DELETE requests with a body, which handles this correctly.\n\n## Changes\n\n- `src/client.ts` — `delete()` method uses `fetch()` when body is present

jmiller added 1 commit 2026-06-20 19:58:46 +00:00

fix: use fetch for DELETE+body to fix file_delete (#649) ... b469a7da13

node:https drops the request body on DELETE requests through some
proxy/TLS configurations, causing Gitea to see an empty body and
return "file already exists" (routes to CREATE handler).

Switch to built-in fetch() for DELETE requests with a body, which
handles this correctly.

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

jmiller merged commit 57b4e5d5ae into main 2026-06-20 20:01:47 +00:00

jmiller deleted branch fix/649-file-delete-body 2026-06-20 20:01:48 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

breaking-change

Breaking API or behavior change

ci-cd

CI/CD pipeline changes

config

Configuration changes

dependencies

Dependency updates

deploy-failure

Deployment failed

docker

Docker/container changes

documentation

Documentation changes

good first issue

Good for newcomers

health-check

Repo health check result

help wanted

Extra attention needed

javascript

JavaScript code

mokostandards

Related to MokoStandards framework

push-failure

Git push operation failed

security

Security vulnerability or hardening

size/l

200-500 lines changed

size/m

50-200 lines changed

size/s

10-50 lines changed

size/xl

500+ lines changed

size/xs

< 10 lines changed

standards-drift

Deviates from MokoStandards

standards-update

MokoStandards compliance update

sync-failure

Sync or mirror failed

typescript

TypeScript code

work-in-progress

Draft or incomplete work

No labels

Priority -

Type -

Milestone

No items

No Milestone

Assignees

Clear assignees

jmiller (Jonathan Miller)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: MokoConsulting/mcp-mokogitea-api#20