Skip to content

[BUGFIX] Mass update script: Avoid passing in incident ids via query parameter, which can cause 414 errors #137

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[BUGFIX] Mass update script: Avoid passing in incident ids via query parameter, which can cause 414 errors #137

wants to merge 1 commit into from

Conversation

akosasante
Copy link
Contributor

@akosasante akosasante commented Sep 25, 2025 •
edited by atlassian bot
Loading

Link back to Jira ticket:
INCIDENTS-2776

Description

During some internal testing of the mass-update script to do bulk resolves, we found that the i / incident-ids flag can cause the script to fail with a 414 error:

<html>
<head><title>414 Request-URI Too Large</title></head>
<body>
<center><h1>414 Request-URI ...
<html>
<head><title>414 Request-URI Too Large</title></head>
<body>
<center><h1>414 Request-URI Too Large</h1></center>
<hr><center>nginx</center>
</body>
</html>

This was when passing in a large number of incidents (100+) into the i flag. The cause was that these incident ids were being appended as query parameters, which eventually created a URL that's too large for the server to handle.

Implementation

Changed the setup steps that occur for the resolve action, to simply pass the parsed list of incident ids to the initial PUT request that's used to fetch incident details; rather than appending them to the global PARAMETERS value that's used to build the query parameters for subsequent API calls.

Steps to test changes

  • Run the mass update script with the resolve action, for a large number of incidents, eg: python mass_update_incidents.py -k mysecretapikey -s myserviceid -a resolve -e "from_email" -i incidentid1,incidentid2,etc

Documentation

- [] There is documentation that needs to be updated upon merging these changes

Links to any documentation to be updated:

@akosasante akosasante changed the title Avoid passing in incident ids via query parameter, which can cause 414 errors [BUGFIX] Mass update script: Avoid passing in incident ids via query parameter, which can cause 414 errors Sep 25, 2025
@akosasante akosasante marked this pull request as ready for review September 26, 2025 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JosefGoodyear JosefGoodyear Awaiting requested review from JosefGoodyear

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@akosasante