HTTPResponse subclasses don't accept base class parameters

[hp1337@…]

I would like to pass content_type when creating a new HttpResponseNotAllowed object. Unfortunately, it does not accept parameters from its base class, HttpResponse. Doing the following results in an error:

return HttpResponseNotAllowed(content_type="application/json", permitted_methods=['POST'], )

There is a simple fix to this. See my patch below. This patch should be applied to other HttpResponse sub-classes as well. Namely, HttpResponseRedirect and HttpResponsePermanentRedirect.

class HttpResponseNotAllowed(HttpResponse):
    status_code = 405

    def __init__(self, permitted_methods, *args, **kwargs):
        super(HttpResponseNotAllowed, self).__init__(*args, **kwargs)
        self['Allow'] = ', '.join(permitted_methods)

[Preston Holmes]

Can you provide more detail about your use case - while I can't find anything in a quick review of the spec that prohibits setting a content-type for a 405 response, it is hard to imagine a client that would do something specific with it.

If this is in fact something that should be supported - it should be supported for most (all?) included subclasses of HttpResponse.

[Claude Paroz]

I also think that we should allow HttpResponse standard parameters for all subclasses. I don't see anything in the HTTP spec that prevents a response to have attached content (with content-type).

[Claude Paroz]

Allow HttpResponse arguments for subclasses

[anonymous]

Sorry, I'm a bit late. My use case is a proprietary RESTful client that will reject all communication that is not of a specific content-type. Even if the status code is correct.

[Claude Paroz <claude@…>]

In [44c09de555f83b0596a0886e3f51a9a84368d036]:

Fixed #18678 -- HttpResponse init arguments allowed for subclasses

Thanks hp1337@… for the report.