Skip to content

Private issue tracker search results#7384

Open
labkey-martyp wants to merge 2 commits intorelease25.7-SNAPSHOTfrom
25.7_fb_private_issue_search_fix
Open

Private issue tracker search results#7384
labkey-martyp wants to merge 2 commits intorelease25.7-SNAPSHOTfrom
25.7_fb_private_issue_search_fix

Conversation

@labkey-martyp
Copy link
Contributor

@labkey-martyp labkey-martyp commented Feb 3, 2026

Rationale

Searching for tickets in a private issue tracker throws after the first result because the search is using a limited user specifically for search, which does not meet the permission check of the private issue tracker. Adding the false flag to not throw on restricted failure will return the full search results but the user will only be able to click into the issues they meet the permission requirements on. This is similar to the list of issues.

Here's the stack trace. This is just when listing the search matches. Since it throws after the first one, no more than one match is ever found.

    org.labkey.api.view.UnauthorizedException:` This issue is in a restricted issue list. You do not have access to this issue
at org.labkey.issue.model.IssueManager.getIssue(IssueManager.java:230)
at org.labkey.issue.model.IssueManager.getIssue(IssueManager.java:192)
at org.labkey.issue.model.IssueManager.resolve(IssueManager.java:1308)
at org.labkey.issue.model.IssueManager$1.resolve(IssueManager.java:1011)
at org.labkey.search.model.AbstractSearchService.resolveResource(AbstractSearchService.java:801)
at org.labkey.jsp.compiled.org.labkey.search.view.search_jsp.getActions(search_jsp.java:93)
at org.labkey.jsp.compiled.org.labkey.search.view.search_jsp._jspService(search_jsp.java:827)
at org.labkey.api.view.JspView.renderView(JspView.java:142)

Changes

  • Add false for throwOnRestrictedFailure option.

Copy link
Contributor

@labkey-klum labkey-klum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had to add the same flag to getCustomSearchResult as well.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants