Skip to content

GML-2131: Respect configured TigerGraph ports during UI login#44

Open
OmarAyo1 wants to merge 1 commit into
tigergraph:mainfrom
OmarAyo1:GML-2131-ui-login-tg-ports
Open

GML-2131: Respect configured TigerGraph ports during UI login#44
OmarAyo1 wants to merge 1 commit into
tigergraph:mainfrom
OmarAyo1:GML-2131-ui-login-tg-ports

Conversation

@OmarAyo1

@OmarAyo1 OmarAyo1 commented Jun 18, 2026

Copy link
Copy Markdown

User description

PR Type

Bug Fix


Description

  • Respect configured TigerGraph gsPort and restppPort during UI login

  • Fix /ui/ui-login incorrectly falling back to pyTigerGraph default RESTPP port 9000

  • Apply the same connection port configuration to both username/password and API-token login paths

  • Prevent valid TigerGraph credentials from being rejected when TigerGraph is exposed on a non-default port such as 14240


Diagram Walkthrough

flowchart LR
  browser["GraphRAG UI login"] --> api["/ui/ui-login"]
  api --> auth["auth()"]
  auth --> config["db_config"]
  config --> ports["gsPort / restppPort"]
  ports --> tg["TigerGraphConnection"]
  tg --> graphs["listGraphs() succeeds"]

File Walkthrough

Relevant files

Bug Fix 1 file

graphrag/app/routers/ui.py
Pass configured TigerGraph gsPort/restppPort into UI login TigerGraphConnection calls


Validation

  • Reproduced the issue with TigerGraph exposed on port 14240

  • Confirmed TigerGraphConnection(..., gsPort=14240, restppPort=14240) succeeds with the same credentials

  • Confirmed the existing /ui/ui-login path fails because it falls back to port 9000

  • Confirmed /ui/ui-login returns 200 OK after passing configured gsPort and restppPort


PR Type

Bug fix


Description

  • Pass configured ports to UI login

  • Fix non-default TigerGraph port authentication


Diagram Walkthrough

flowchart LR
  login["UI login credentials"] 
  auth["auth() connection setup"]
  config["Configured gsPort/restppPort"]
  tg["TigerGraphConnection"]
  graphs["listGraphs()"]

  login -- "calls" --> auth
  config -- "provides ports" --> auth
  auth -- "creates" --> tg
  tg -- "authenticates via configured ports" --> graphs
Loading

File Walkthrough

Relevant files
Bug fix
ui.py
Configure TigerGraph ports for UI authentication                 

graphrag/app/routers/ui.py

  • Adds gsPort from db_config to UI login connections
  • Adds restppPort from db_config to UI login connections
  • Applies port configuration to token and password login paths
+10/-3   

@tg-pr-agent

tg-pr-agent Bot commented Jun 18, 2026

Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Default Ports

Validate that passing None from db_config.get("gsPort") or db_config.get("restppPort") does not override pyTigerGraph constructor defaults when these keys are absent. If pyTigerGraph does not handle None, omit the arguments unless configured.

conn = TigerGraphConnection(
    host=db_config["hostname"],
    gsPort=db_config.get("gsPort"),
    restppPort=db_config.get("restppPort"),

Comment thread graphrag/app/routers/ui.py
@OmarAyo1 OmarAyo1 force-pushed the GML-2131-ui-login-tg-ports branch from 01d1bc0 to 2319b01 Compare June 18, 2026 21:44
@OmarAyo1 OmarAyo1 force-pushed the GML-2131-ui-login-tg-ports branch from 2319b01 to 59ab0aa Compare June 18, 2026 21:48
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.

1 participant