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.

First 3 of 3 Posts Switch mode

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 91898 914
New
AstonJ
The obligatory hello world thread! Who are you and where are you from? :stuck_out_tongue:
4616 55835 594
New
byu
@chrismccord : I just saw the Extract AGENTS.md from Phoenix.new into phx.new generator commit to the phoenix project. My initial shotgu...
New
arcanemachine
I was working on an Ecto migration and I needed a timestamp. So, for the nth time, I looked up the different data types for timestamps, a...
New
alexslade
Fly’s CEO posted this recently - Turn And Face The Strange · The Fly Blog It says that Fly is going all-in on sprites, which is a worry ...
New
Herve37
We’re evaluating API mocking tools for OpenAPI-based projects and would love to hear what other teams are using. We’re particularly inte...
New
matt-savvy
Is there a word for the ~> symbol used in Version strings? Do you also just call it a Squiggle Arrow™ ?!
New

Other Trending Topics Top

JesseHerrick
Hey, I’m Jesse and I’m the main contributor behind Dexter, a full-featured, lightning-fast Elixir LSP optimized for large codebases. It s...
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
mcass19
ExRatatui lets you cook up rich terminal UIs in Elixir, powered by Rust’s ratatui via Rustler NIFs. Build interactive terminal applicatio...
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
netoum
Corex is an accessible, unstyled UI component library for Phoenix that integrates Zag.js state machines using Vanilla JavaScript and Live...
New
ausimian
Emily is an Elixir library that runs Nx computations on Apple’s MLX. Install it as the default Nx backend and Nx, defn, Axon, Nx.Serving,...
New

We're in Beta

About us Mission Statement