Skip to content

doc-auth-token-url-variables - #150

Closed
general-kroll-4-life wants to merge 1 commit into
mainfrom
bugfix/server-dynamics
Closed

general-kroll-4-life wants to merge 1 commit into
mainfrom
bugfix/server-dynamics

Conversation

@general-kroll-4-life

Copy link
Copy Markdown
Contributor

Summary:

  • Stop presenting document-registered auth as a runtime token_url override; omnisdk now overlays the document token URL and resolves config.auth.variables when no --auth is supplied.
  • Preserve explicit token_url overrides from --auth and provider aliases.
  • Declare client credential environment variables in the endpoint selector test document and allow selector scenarios to run without --auth.
  • Added robot test Document Auth Applies Token URL Variables.

Fixes #890

Description

Type of change

  • Bug fix (non-breaking change to fix a bug).
  • Feature (non-breaking change to add functionality).
  • Breaking change.
  • Other (eg: documentation change). Please explain.

Issues referenced.

Evidence

Checklist:

  • A full round of testing has been completed, and there are no test failures as a result of these changes.
  • The changes are covered with functional and/or integration robot testing.
  • The changes work on all supported platforms.
  • Unit tests pass locally, as per the developer guide.
  • Robot tests pass locally, as per the developer guide.
  • Linter passes locally, as per the developer guide.

Variations

Tech Debt

Summary:

- Stop presenting document-registered auth as a runtime `token_url` override; omnisdk now overlays the document token URL and resolves `config.auth.variables` when no `--auth` is supplied.
- Preserve explicit `token_url` overrides from `--auth` and provider aliases.
- Declare client credential environment variables in the endpoint selector test document and allow selector scenarios to run without `--auth`.
- Added `robot` test `Document Auth Applies Token URL Variables`.

Fixes #890
@general-kroll-4-life
general-kroll-4-life deleted the bugfix/server-dynamics branch November 5, 2026 09:36
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