Skip to content

fix: remove deprecated GitHubOAuth connector creation from apply-extras#228

Merged
RobiladK merged 3 commits into
mainfrom
fix/remove-deprecated-github-oauth-connector
Jun 22, 2026
Merged

fix: remove deprecated GitHubOAuth connector creation from apply-extras#228
RobiladK merged 3 commits into
mainfrom
fix/remove-deprecated-github-oauth-connector

Conversation

@dm-chelupati

Copy link
Copy Markdown
Collaborator

Problem

The platform deprecated the GitHubOAuth connector type in build 26.4.216.0 (April 2026). The connector status endpoint now hardcodes it as Disconnected with message:

This OAuth connector type is deprecated, please remove.

This causes the portal to show a permanent "github (Disconnected)" ghost row that cannot be deleted via any API.

Root Cause

apply-extras.sh and Apply-Extras.ps1 were calling PUT /api/v2/extendedAgent/connectors/github with dataConnectorType: GitHubOAuth — a type the platform now rejects.

Fix

  • Remove PUT /api/v2/extendedAgent/connectors/github from both bash and PowerShell
  • Use PUT /api/v2/github/domains/github_com for the optional GITHUB_PAT path
  • Update OAuth status check from /api/v1/Github/auth/status to /api/v2/github/domains
  • Update OAuth config URL from /api/v1/Github/config to /api/v2/github/oauth/config
  • Remove fallback check against deprecated connector list
  • Keep OAuth browser flow and repo wiring unchanged

Testing

Deployed a new agent (dt-v2) using the dynatrace-servicenow recipe with this fix. GitHub OAuth completed successfully, repo was wired, and no ghost "Disconnected" connector appears in the portal.

The platform deprecated the GitHubOAuth connector type in build 26.4.216.0
(April 2026). The connector status endpoint now hardcodes it as 'Disconnected'
with message 'This OAuth connector type is deprecated, please remove.'

The actual GitHub auth is stored via /api/v2/github/domains (OAuth flow or PAT).
Code repos (/api/v2/repos/{name}) use that auth directly without needing a
separate connector record.

Changes:
- Remove PUT /api/v2/extendedAgent/connectors/github (both bash and PS1)
- Use PUT /api/v2/github/domains/github_com for GITHUB_PAT path
- Update OAuth status check from v1 auth/status to v2 github/domains
- Remove fallback check against deprecated connector list
- Keep OAuth browser flow and repo wiring unchanged
The ParseFile() return value (AST) was being sent to stdout,
causing the syntax check to report every .ps1 file as failed.
Adding $null = suppresses the output so only actual parse
errors are reported.
…connectors

- assemble-agent.sh: .md files in data/ now go into 'knowledge' array
  (uploaded via /api/v1/AgentMemory/upload) instead of 'knowledgeItems'
  (which creates ARM KnowledgeText connectors)
- export-agent.sh: KnowledgeText items exported as plain .md files in
  data/ so they round-trip correctly through assemble → deploy
@RobiladK RobiladK merged commit fa0d92e into main Jun 22, 2026
2 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