)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":2475,"name":"Antoine Musso","email":"hashar@free.fr","username":"hashar"},"change_message_id":"50f8495d359e533f0d2c725845222f5c66324c97","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"481b9f55_cfb09ee9","updated":"2025-08-29 12:55:16.000000000","message":"The `allow_redirects` should be passed directly to the `requests.Request` object which is prepared for creating or deleting a node.  That saves some repetitions from `jenkins_request` and `jenkins_open`.\n\nI would expect the view/jobs etc to have the same issue?  I have tested node deletion and does a 302 redirect to /computer/, I am not sure why that would end up being an error?","commit_id":"fcfab722864f730bc44206e432c1ced6b7b5da26"}],"jenkins/__init__.py":[{"author":{"_account_id":2475,"name":"Antoine Musso","email":"hashar@free.fr","username":"hashar"},"change_message_id":"50f8495d359e533f0d2c725845222f5c66324c97","unresolved":true,"context_lines":[{"line_number":1643,"context_line":"        \u0027\u0027\u0027"},{"line_number":1644,"context_line":"        self.get_node_info(name)"},{"line_number":1645,"context_line":"        self.jenkins_open("},{"line_number":1646,"context_line":"            requests.Request(\u0027POST\u0027, self._build_url(DELETE_NODE, locals())),"},{"line_number":1647,"context_line":"            allow_redirects\u003dFalse,"},{"line_number":1648,"context_line":"        )"},{"line_number":1649,"context_line":"        if self.node_exists(name):"}],"source_content_type":"text/x-python","patch_set":1,"id":"042a9574_c8f07099","line":1646,"range":{"start_line":1646,"start_character":12,"end_line":1646,"end_character":76},"updated":"2025-08-29 12:55:16.000000000","message":"The node deletion is a POST created from a `requests.Request()` which we can be passed `allow_redirects\u003dFalse`.  I don\u0027t think it should be exposed in `jenkins_request` and `jenkins_open`.\n\nI am not sure why the creation of a node would be redirect to a page that errors out?\n\nWhen deleting a node, once the node is deleted it does a 302 redirect to `/computer`, I am no sure how it would redirect to a page that 403/404?\n\nI would expect the deletion of a job to suffer from the same issue and would need a similar adjustment.","commit_id":"fcfab722864f730bc44206e432c1ced6b7b5da26"}]}
