Allow default redirect status value to be configured

[wout]

I'm using htmx in two apps now, and requests from hx-boost require 303 redirects after create/update/delete. Right now, I'm passing a modified status everywhere, which is tedious and error-prone. It would be great to be able to define a redirect status globally in the Lucky config.

[jwoertink]

I haven't messed with this yet, but do you use the Action's redirect("/whatever", status: 303) method? It looks like there's no global default right now, just default values in the methods

lucky/src/lucky/redirectable.cr

Line 121 in 10752ad

def redirect(to path : String, status : Int32 = 302) : Lucky::TextResponse

We could make a habitat config in this module though and default it, then you could globally override it. Or did you have something in particular in mind for this?

[wout]

Yeah, I'm using status: 303 everywhere because 302 results in an eternal redirect loop. I mentioned it on the htmx discord, and they told me 303 is the "correct" way in CREATE/PUT/DELETE situations where the result is rendered elsewhere, but that's probably also an opinion. 😏️ There's also a of discussion about the redirect status code in the Turbo/Rails community by the looks of it, so being able to configure such a thing globally wouldn't hurt.

I think a Habitat setting would be perfect, and that's exactly what I had in mind. I'm now monkey-patching the redirectable module, but that feels a bit dirty.

[jwoertink]

Cool, yeah that sounds good. We still have time to get it in the next release. We're blocked on some other stuff, so if you wanted to get that up, we can add that in.

[wout]

Great. I'll prepare a PR tomorrow!

[wout]

Implemented with #1838.