Skip to content

fix(transport-commons): map HEAD requests like GET for REST services - #3701

Merged
daffl merged 1 commit into
feathersjs:dovefrom
rome-xi:contrib/feathers-3464
Sep 11, 2026
Merged

daffl merged 1 commit into
feathersjs:dovefrom
rome-xi:contrib/feathers-3464

Conversation

@rome-xi

@rome-xi rome-xi commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Fixes #3464

Map HEAD like GET in getServiceMethod so REST HEAD probes do not throw MethodNotAllowed. Unit tests cover HEAD/head with and without an id.

Signed-off-by: rome-xi <rome-xi@users.noreply.github.com>

@daffl daffl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes sense, thank you!

@daffl
daffl merged commit fcbf602 into feathersjs:dove Sep 11, 2026
2 checks passed
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.

Calls to feathers services with HEAD method are throwing an error since v5.x.

2 participants