brucepomeroy

brucepomeroy

Controllers generated with mix phx.gen.json respond with 422 status for changeset errors, whereas HTML controllers, generated with mix phx.gen.html respond with 200. I assume this is intentional, curious about the reasoning behind it. I always thought it was conventional to respond with 4xx status in the case of invalid input.

I’ve been using Unpoly, a JS library that merges HTML responses into the dom of the existing page (similar in concept to Turbolinks). Unpoly expects 4xx responses for invalid form submissions so I’ve been modifying my controllers to provide 422 responses. I’m curious why the generators don’t do this by default.

Showing Posts 1 to 3

zimt28

zimt28

I’d argue that the status responses on the api are about the data or the operation whereas the http status on the webpage refers to the page. When a changeset error occurs while using a web UI, you’re still getting the right page, so 200 (IMO) is the right status code. If the error occurs during an api call, the error message tells you that the operation failed due to a client error (4xx), so that makes sense as well :slight_smile:

Edit: Just a thought, I can totally see that returning the same error messages in both cases makes sense too. However, I still think that 95+ % of all web apps don‘t have a reason to return the right status code and as it can be added easily (you could even define your own generator templates), it shouldn‘t be set by default.

brucepomeroy

brucepomeroy OP

Thanks zimt28. Yeah I see what you mean. I’m still on the fence about which is more “correct”. Regardless, as you say, it’s trivial to change the response code to whatever works best for your situation. Thanks again.

jeremyjh

jeremyjh

I think 422 makes sense for web pages as well. The POST operation was in fact, invalid and the desired operation was not completed. Yes there is some HTML to look at, but there is really no semantic difference between these two cases, only a different content-type.

— All posts loaded —

Where Next? Top

Trending in Discussions Top

AstonJ
As the title says, please share what you’ve been up to with Elixir. Whether that’s been learning it, looking into it, making stuff with i...
2977 92995 915
New
caslu
I want to open this thread for you all to discuss and help those who really like Ash but are still hesitant to use it in a real project. ...
New
mudasobwa
I am happy to introduce the very α version of the new programming language compiled to BEAM. Welcome Cure. It has literally three kille...
New
GES233
I’m posting this in response to Jose’s recent tweet (Cr. link) : People are sleeping on Elixir for a coding harness: Hot-code swappi...
New
marciol
It would be helpful to have a list of companies worldwide that hire engineers without prior experience in Elixir. Often, it can be quite ...
New
durvia
Anyone running long-lived stateful processes on BEAM? We’re building an AI agent runtime and would love to compare notes. We’re a small ...
New
nseaSeb
I’ve just put together a small POC exploring PDF inspection from Elixir/Phoenix: The idea is pretty simple: drag & drop a PDF in a...
New

Other Trending Topics Top

garrison
Hobbes is a low-level distributed database for the Elixir programming language. Hobbes provides a simple, safe, and scalable storage lay...
New
marciok
Hi there! We created Gust: A task orchestrator inspired by Airflow. For those who have never heard about Aiflow, it’s a Python-based wor...
New
jimsynz
Beam Bots (or just BB for short) is a framework for building fault-tolerant robotics applications in Elixir using familiar OTP patterns. ...
New
Dmk
Xamal is a deployment tool for Elixir apps that deploys native releases to bare metal servers over SSH. It’s a port of GitHub - basecamp/...
New
Damirados
Hello everyone. After busy few months I am happy to announce v0.1.0 of Emerge & Solve. They are GUI (Emerge) and State management (S...
New
wintermeyer
There are three potential reasons for members of this forum to have a look at https://vutuv.de You are tired or annoyed of LinkedIn. Yo...
New

We're in Beta

About us Mission Statement

Options

Thread Display Mode




Thread Preview

Skip Thread Previews