Skip to content

feat(goTo): add postAuth and modules#2947

Open
immdipu wants to merge 2 commits into
mainfrom
goto-postAuth
Open

feat(goTo): add postAuth and modules#2947
immdipu wants to merge 2 commits into
mainfrom
goto-postAuth

Conversation

@immdipu
Copy link
Copy Markdown
Contributor

@immdipu immdipu commented May 1, 2026

  • Added postAuth for GoTo. We need it to fetch the accountKey from /admin/rest/v1/me, GoTo API endpoints require it.
  • Split GoTo into two modules: goTo (api.getgo.com, default) and goToConnect (api.goto.com), since the two share auth but live on different base URLs.

Copilot AI review requested due to automatic review settings May 1, 2026 07:06
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds post-authentication metadata retrieval to the GoTo connector (to discover accountKey) and updates the GoTo provider catalog to support two modules with different base URLs, along with a manual smoke test program.

Changes:

  • Introduce providers/goto connector implementation with GetPostAuthInfo() that fetches accountKey from /admin/rest/v1/me.
  • Update providers/goTo.go catalog entry to require post-auth info and define goTo + goToConnect modules.
  • Add test/goto helper + test/goto/auth-metadata manual smoke test to print the retrieved accountKey.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
test/goto/connector.go Adds a test helper to construct an authenticated GoTo connector instance from local creds.
test/goto/auth-metadata/main.go Adds a manual go run smoke test to call GetPostAuthInfo() and log the accountKey.
providers/goto/connector.go Defines the GoTo connector struct and NewConnector() using internal/components.
providers/goto/authmetamodel.go Adds a small model helper for accountKey post-auth catalog vars.
providers/goto/authmetadata.go Implements post-auth discovery call to /admin/rest/v1/me and returns CatalogVars.
providers/goTo.go Updates the GoTo ProviderInfo to require post-auth metadata and introduces module definitions.

Comment thread providers/goto/authmetadata.go
Comment thread test/goto/auth-metadata/main.go
Comment thread test/goto/connector.go
Comment thread providers/goto/authmetadata.go
Comment thread providers/goto/authmetadata.go
Comment thread providers/goto/authmetadata.go
@immdipu immdipu requested a review from RajatPawar May 1, 2026 08:04
@immdipu immdipu self-assigned this May 1, 2026
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