Skip to content

fix(jira): require a query for issue search - #129619

Draft
sentry-junior[bot] wants to merge 1 commit into
josh/typing-strict-types-onlyfrom
josh/typing-stack/jira-search-query-required
Draft

sentry-junior[bot] wants to merge 1 commit into
josh/typing-strict-types-onlyfrom
josh/typing-stack/jira-search-query-required

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Stacked on #129614 (strict mypy, types only). This is #129430 rebased onto it so the fix lands after the typing PR.

A None query reached the Jira clients and failed with an AttributeError or TypeError. Fail at the integration boundary instead, for both Jira Cloud and Jira Server. Both search endpoints already reject empty queries before they call this method. This replaces the cast(str, query) that #129614 kept.

No new test: the existing endpoint tests cover the empty-query guards, and the assertion can't be reached from them.

via @joshuarli.

--

View Session [Sentry]

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 3, 2026
@sentry-junior
sentry-junior Bot force-pushed the josh/typing-strict-types-only branch from 9fda5d0 to 73fa505 Compare November 3, 2026 00:31
Co-Authored-By: Joshua Li <14209781+joshuarli@users.noreply.github.com>
@sentry-junior
sentry-junior Bot force-pushed the josh/typing-stack/jira-search-query-required branch from 7cbbba9 to 43d1e56 Compare November 3, 2026 00:34

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant