electricdusk
/
rushlink
3
3
Fork 2
Code Issues 21 Pull Requests 1 Releases Activity
Labels Milestones
New Issue

Bad Accept header results in internal server error #66

Closed
opened 3 years ago by electricdusk · 2 comments
electricdusk commented 3 years ago
Owner

See https://hashru.link/Qd_z.

In this case, the client (presumably) provided a bad Accept header. In this case, rushlink should error with an 400 Bad Request error. However, it fails with an internal server error.

See <https://hashru.link/Qd_z>. In this case, the client (presumably) provided a bad `Accept` header. In this case, rushlink should error with an `400 Bad Request` error. However, it fails with an internal server error.
electricdusk added the bug needs-test good-beginner-bug labels 3 years ago
electricdusk commented 3 years ago
Poster
Owner

This error seems to occur when the client's Accept value has a leading white space. By the spec, this should be trimmed however.

This error seems to occur when the client's `Accept` value has a leading white space. By the spec, this should be trimmed however.
electricdusk commented 3 years ago
Poster
Owner

However, a media-range value '*' is still invalid, I think.

However, a `media-range` value `'*'` is still invalid, I think.
electricdusk closed this issue 3 years ago
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
master
sql
Labels
Apply labels
Clear labels
bug
Uninteded behaviour feature
Request for a feature to be implemented good-beginner-bug
Bugs are probably easy to fix needs-test
Needs a regression test question
Question about the software wontfix
Wont be resolved
No Label bug feature good-beginner-bug needs-test question wontfix
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: electricdusk/rushlink#66
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch '%!s(<nil>)'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes