Skip to content

fix: use dedicated fragments for AGW token generation#110

Draft
mheidinger wants to merge 2 commits into
SAP:mainfrom
mheidinger:agw-new-token-fetch
Draft

fix: use dedicated fragments for AGW token generation#110
mheidinger wants to merge 2 commits into
SAP:mainfrom
mheidinger:agw-new-token-fetch

Conversation

@mheidinger
Copy link
Copy Markdown
Contributor

@mheidinger mheidinger commented May 12, 2026

Description

Adding token fetching based on dedicated AGW fragments for the AGW client. Additionally it exposes the URL of AGW as fetched from the fragment.

Will mark as ready once tested

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Dependency update

How to Test

  1. Use an agent that has the new fragment generation applied
  2. Make sure that these fragments were created for your test tenant
  3. Do a regression test of the existing client functionality
  4. Also test whether the standalone auth fetching functions work and return valid tokens + URLs

Checklist

Before submitting your PR, please review and check the following:

  • I have read the Contributing Guidelines
  • I have verified that my changes solve the issue
  • I have added/updated automated tests to cover my changes
  • All tests pass locally
  • I have verified that my code follows the Code Guidelines
  • I have updated documentation (if applicable)
  • I have added type hints for all public APIs
  • My code does not contain sensitive information (credentials, tokens, etc.)
  • I have followed Conventional Commits for commit messages

@mheidinger mheidinger changed the title Use dedicated fragments for AGW token generation and expose these fix: use dedicated fragments for AGW token generation May 12, 2026
@mheidinger mheidinger force-pushed the agw-new-token-fetch branch from 24ed4be to 39c46a7 Compare May 12, 2026 14:17
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