Skip to content

Add support for user-configurable Git hosts & repo lists for internal/external issue search #1110#1126

Merged
smlambert merged 1 commit intoadoptium:masterfrom
SehrishHussain:feature/user-configurable-repos-1110
Dec 11, 2025
Merged

Add support for user-configurable Git hosts & repo lists for internal/external issue search #1110#1126
smlambert merged 1 commit intoadoptium:masterfrom
SehrishHussain:feature/user-configurable-repos-1110

Conversation

@SehrishHussain
Copy link
Contributor

This PR enhances the getInternalGitIssues API to support user configurable GitHub style hosts and dynamic repository lists defined in a config file (trssConf.json).
This allows TRSS to search issues across multiple Git providers, including GitHub Enterprise instances (e.g., github.ibm.com) and public GitHub, without requiring code changes.

The frontend (PossibleIssues.jsx) was updated accordingly to consume the new API.

Motivation

Previously, TRSS only supported a hard coded GitHub Enterprise search.
The request in issue #1109 asked for:

  • User configurable repo sources
  • Future support for multiple vendors
  • A dedicated config file to store repos and tokens

This PR implements that.

@netlify
Copy link

netlify bot commented Nov 22, 2025

Deploy Preview for eclipsefdn-adoptium-trss canceled.

Name Link
🔨 Latest commit d011856
🔍 Latest deploy log https://app.netlify.com/projects/eclipsefdn-adoptium-trss/deploys/6921f7a67649390008840ede

@SehrishHussain
Copy link
Contributor Author

@llxia and @smlambert can you check the PR and provide your feedback.

@SehrishHussain SehrishHussain force-pushed the feature/user-configurable-repos-1110 branch from c20a588 to d011856 Compare November 22, 2025 17:49
@karianna karianna requested review from llxia and smlambert November 23, 2025 21:22
@SehrishHussain
Copy link
Contributor Author

I tested the enhanced getInternalGitIssues endpoint locally using:

  • trssConf.json with multiple hosts.
  • confirmed that results were merged across all configured repositories.
  • Verified correct behavior when no config file is provided (returns a clear, user-friendly message).
  • Confirmed that only repos listed in the config are queried and returned.
  • Ensured unreachable hosts are skipped gracefully without breaking the overall response.
  • Validated that multiple hosts return a single combined list of issues as expected.

@smlambert
Copy link
Contributor

Thanks @SehrishHussain - I will set up a local server to test your build internally also (with the plan to complete the review and merge by early next week). 👍

Copy link
Contributor

@smlambert smlambert left a comment

Choose a reason for hiding this comment

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

thanks @SehrishHussain and also thanks @annaibm for testing internally to verify that it works for the case captured in the possibleIssueRepos.json file that is part of this PR.

@smlambert smlambert merged commit ce2a988 into adoptium:master Dec 11, 2025
7 checks passed
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