Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

SessionConfig.GitHubToken / GitHubTokenProvider still route to api.github.com instead of the GHEC Data Residency tenant endpoint, even with CopilotClientMode.Empty — same defect class as #4527, unresolved on the SDK session-level path

Đang mở
#4,938 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
45/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
csharp, github-actions, shell

Hướng nghiên cứu

The issue is in the GitHub.Copilot.SDK .NET library, specifically the session authentication code paths for SessionConfig.GitHubToken and GitHubTokenProvider. Start by examining the SDK source for how it resolves the API endpoint for token validation, likely in the Copilot CLI runtime bundled with the SDK. Look for where the tenant host is derived from the token or environment variables, and compare it to the working ambient path (UseLoggedInUser = true). The fix likely involves applying the same endpoint resolution logic used for the ambient path to the session-level credential paths.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

area:authentication area:enterprise area:networking
Describe the bug
Describe the bug

On a GitHub Enterprise Cloud with Data Residency (GHEC-DR) + EMU tenant (<tenant>.ghe.com), the GitHub.Copilot.SDK (.NET) per-session authentication mechanisms — SessionConfig.GitHubToken (stable field) and SessionConfig.GitHubTokenProvider (experimental field) — fail to route Copilot validation calls to the tenant's own endpoint (api.<tenant>.ghe.com / copilot-api.<tenant>.ghe.com). Instead they fall back to the public api.github.com, which rejects the tenant's gho_ token with 401 Bad credentials.

This is the same defect class as #4527 ("copilot -p fails with 401 on GHEC data residency ... model-catalog fetch hits api.githubcopilot.com instead of the tenant endpoint"), which was fixed for the CLI's non-interactive prompt-mode path. The fix does not appear to cover the SDK's session-level credential-forwarding path used for multi-user/server integrations, which is the officially documented pattern for this exact use case (see SDK README "Advanced Usage" / GitHubTokenProvider docs).

Steps to reproduce the behavior
  1. Authenticate a web user against <tenant>.ghe.com via a standard OAuth Authorization Code flow (separate from copilot login), obtaining a valid gho_ user access token scoped to the tenant.
  2. Create a client explicitly for the documented multi-user/server pattern:
    var client = new CopilotClient(new CopilotClientOptions
    {
        Mode = CopilotClientMode.Empty,
        BaseDirectory = "<isolated temp dir>",
        UseLoggedInUser = false
        // no GitHubToken set at the client level
    });
    await client.StartAsync();
    
  3. Create a session using either documented per-session mechanism:
    var session = await client.CreateSessionAsync(new SessionConfig
    {
        Model = "gpt-5",
        GitHubToken = userToken, // stable field
        AvailableTools = new ToolSet().AddBuiltIn(BuiltInTools.Isolated) // required by Mode.Empty
    });
    
    or
    var session = await client.CreateSessionAsync(new SessionConfig
    {
        Model = "gpt-5",
        GitHubTokenProvider = args => Task.FromResult(GitHubTokenProviderResult.FromToken(new GitHubToken
        {
            AccessToken = userToken,
            ExpiresIn = 8 * 60 * 60
        })),
        AvailableTools = new ToolSet().AddBuiltIn(BuiltInTools.Isolated)
    });
    
  4. Send any prompt.
What the user sees

SessionConfig.GitHubToken: CreateSessionAsync throws with:

Communication error with Copilot CLI: Request session.create failed with message:
SDK session authentication failed: Failed to fetch Copilot user info: 401 Unauthorized: {"message":"Bad credentials"}

Confirmed (via process-level environment variable injection of GITHUB_HOST, GH_HOST, GITHUB_API_URL set to the tenant host, merged with the full parent environment rather than replacing it) that this call always targets https://api.github.com/copilot_internal/user, regardless of these variables. The token is a valid tenant token — it simply cannot be valid against api.github.com.

SessionConfig.GitHubTokenProvider: CreateSessionAsync throws with a local, pre-network validation error instead:

ErrorType=query Message=Execution failed: InvalidArg, No GitHub OAuth token or Copilot HMAC key provided

No outbound network call is observed for this path at all — the provided callback does not appear to be invoked before this error fires, regardless of CopilotClientMode.

Expected behavior

Per the SDK's own documentation, both SessionConfig.GitHubToken and SessionConfig.GitHubTokenProvider should let the runtime resolve the Copilot API base URL from the tenant associated with the supplied token (api.<tenant>.ghe.com / copilot-api.<tenant>.ghe.com), exactly as the ambient UseLoggedInUser = true path already does successfully for the same tenant/token combination.

What we ruled out
Attempt Result
UseLoggedInUser = true (ambient identity, same tenant/account) Works — proves token/tenant/account are valid
GITHUB_HOST / GH_HOST / GITHUB_API_URL env vars on the client process (merged with the full parent environment, not replacing it) No effect — still routes to api.github.com
CopilotClientMode.Empty with BaseDirectory and AvailableTools set (both required, undocumented prerequisites discovered via trial/error) Session construction succeeds, but the auth call still mis-routes
Different token sources (OAuth Authorization Code flow token, copilot-cli token extracted from Windows Credential Manager) Identical failure for both
Additional context

Same host mis-routing defect class as #76 (fixed in v0.0.332) and #4527 (fixed, but only for the CLI -p prompt-mode code path). Possibly related to #4378 (MCP registry policy fetch, same GHEC-DR fallback pattern, different subsystem). This report is scoped specifically to the GitHub.Copilot.SDK .NET session-level credential APIs (SessionConfig.GitHubToken, SessionConfig.GitHubTokenProvider), which does not appear to be covered by any of the above fixes as of CLI 1.0.84-5 (bundled with SDK 1.0.15-preview.0).

This blocks the documented multi-user/server integration pattern (one shared runtime serving many end users, each with their own GitHub Copilot entitlement) for any GHEC Data Residency tenant — the SDK's flagship use case for SessionConfig.GitHubToken/GitHubTokenProvider.

Suggested fix: apply the same per-subscription/tenant endpoint resolution used by the ambient (UseLoggedInUser) path to the SessionConfig.GitHubToken and GitHubTokenProvider code paths — likely by deriving the tenant host from the token itself rather than defaulting to api.github.com.

Affected version
GitHub.Copilot.SDK (.NET NuGet package): 1.0.15-preview.0
Bundled/pinned Copilot CLI runtime: 1.0.84-5 (older than 1.0.85, referenced as fixing #4527)
Environment
  • Host tenant type: GitHub Enterprise Cloud with Data Residency + Enterprise Managed Users (confirmed via "plan":{"name":"emu_user"} on GET https://api.<tenant>.ghe.com/user)
  • Operating system: Windows 11
  • CPU architecture: x86_64 (AMD64)
  • .NET: 8 (SDK preview 11.0.100-rc.1.26425.128)
  • Auth method: OAuth Authorization Code flow token (separate from copilot login)
Affected version

No response

Steps to reproduce the behavior
  1. On a GHEC Data Residency tenant (<tenant>.ghe.com), obtain a valid gho_ user access token via a standard OAuth Authorization Code flow (not copilot login).

  2. Create a CopilotClient explicitly configured for the documented multi-user/server pattern, with no client-level GitHub credential:

    var client = new CopilotClient(new CopilotClientOptions
    {
        Mode = CopilotClientMode.Empty,
        BaseDirectory = "<isolated temp dir>",
        UseLoggedInUser = false
    });
    await client.StartAsync();
    
  3. Create a session passing the token via the stable SessionConfig.GitHubToken field:

    var session = await client.CreateSessionAsync(new SessionConfig
    {
        Model = "gpt-5",
        GitHubToken = userToken,
        AvailableTools = new ToolSet().AddBuiltIn(BuiltInTools.Isolated)
    });
    
  4. Observe CreateSessionAsync throw:

    SDK session authentication failed: Failed to fetch Copilot user info: 401 Unauthorized: {"message":"Bad credentials"}
    
  5. Repeat step 3 using SessionConfig.GitHubTokenProvider instead:

    var session = await client.CreateSessionAsync(new SessionConfig
    {
        Model = "gpt-5",
        GitHubTokenProvider = args => Task.FromResult(GitHubTokenProviderResult.FromToken(new GitHubToken
        {
            AccessToken = userToken,
            ExpiresIn = 8 * 60 * 60
        })),
        AvailableTools = new ToolSet().AddBuiltIn(BuiltInTools.Isolated)
    });
    
  6. Observe a different, local (pre-network) failure instead:

    Execution failed: InvalidArg, No GitHub OAuth token or Copilot HMAC key provided
    
  7. Confirm the same token/tenant/account works correctly via the ambient path, ruling out a token or tenant configuration issue:

    var client = new CopilotClient(new CopilotClientOptions { UseLoggedInUser = true });
    
  8. (Optional, to rule out host-resolution env vars) Re-run step 2–4 with GITHUB_HOST, GH_HOST, and GITHUB_API_URL set to the tenant host in CopilotClientOptions.Environment (merged with the full parent environment). Observe identical failure — the runtime still calls api.github.com for the Copilot user info check.

Expected behavior

Both SessionConfig.GitHubToken and SessionConfig.GitHubTokenProvider should let the runtime resolve the Copilot API base URL from the tenant associated with the supplied token (api.<tenant>.ghe.com / copilot-api.<tenant>.ghe.com), exactly as the ambient UseLoggedInUser = true path already does successfully for the same tenant/token combination — session creation should succeed and the prompt should return a normal assistant response, without ever contacting api.github.com.

Secondarily:

  • SessionConfig.GitHubTokenProvider should actually invoke the supplied callback (with a GitHubTokenProviderArgs.Host reflecting the resolved tenant host) before failing, rather than failing locally with a generic "no token provided" error that never reaches the callback.
  • If the resolved token is rejected by the tenant's Copilot service, the error should surface the actual host and endpoint that was called (as already suggested for the CLI in #4527), instead of a generic Bad credentials message that gives no indication the wrong host was contacted.
Additional context

No response

Ngôn ngữ chính
Shell
Star
11.2k
Fork
1.9k
Merge trung bình
14 giờ 16 phút
Pull request đã merge (30 ngày)
6

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của github/copilot-cli

Tất cả issue của github/copilot-cli

Issue tương tự

Thêm issue về Shell/Bash

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.