Kong / Kong/insomnia

HTTP Status 303 See Other redirects do not use GET

Open
#3,818 4 comments 5 reactions 0 assignees View on GitHub
B-bug S-unverified
Dominant language
TypeScript
Stars
40k
Forks
2.4k
Avg merge
2d 13h
Merged PRs (30d)
61

Description

**Describe the bug**
When an API returns a 303 See Other status code, Insomnia will follow the redirect with whatever HTTP Method was used for the original request instead of using the GET method as is standard.

**To Reproduce**
Steps to reproduce the behavior:
1. Make a PATCH request to an endpoint that returns a 303 See Other status.
2. See that the request calls the redirect URL with the PATCH method instead of GET.

**Expected behavior**
The expected behavior is for the URL in the Location header to be called using the GET method.

**Screenshots**
If applicable, add screenshots to help explain your problem.

**Desktop (please complete the following information):**
Version: Insomnia 2021.4.0
Release date: 6/29/2021
OS: Darwin x64 19.5.0
Electron: 9.3.5
Node: 12.14.1
V8: 8.3.110.13-electron.0
Architecture: x64
node-libcurl: libcurl/7.73.0 OpenSSL/1.1.1k zlib/1.2.11 brotli/1.0.9 zstd/1.4.9 libidn2/2.1.1 libssh2/1.9.0 nghttp2/1.42.0

Contributor guide

Open the contributing guide

Research direction

No source file or test is named in the issue. Reproduce a PATCH request receiving a 303 response, then trace Insomnia's redirect handling to the method used for the Location URL; done means the redirected request uses GET and the behavior is covered by a regression test.

Written by the indexing model from the issue text.

Assessment

Tech stack
node.js, typescript
Domain
api
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.