Skip to content

Add changelog entry for GRAL-5883 OAuth/auth-header fixes - #311

Open
dmitriyeff wants to merge 1 commit into
masterfrom
gral-5883-changelog
Open

dmitriyeff wants to merge 1 commit into
masterfrom
gral-5883-changelog

Conversation

@dmitriyeff

Copy link
Copy Markdown
Contributor

Summary

  • Documents two sdk-manager template fixes (empty Authorization header alongside API key auth; OAuth refresh running before the access-token check) under Unreleased, since these don't flow through the api-docs-driven SDK changelog automation (that only covers OpenAPI spec changes).

Ref: GRAL-5883

Copilot AI lite review requested due to automatic review settings September 17, 2026 11:04
@dmitriyeff
dmitriyeff requested a review from a team as a code owner September 17, 2026 11:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

This is a low-risk documentation change with only minor formatting and wording nits.

Pull request overview

Documents GRAL-5883 SDK-manager fixes in the Unreleased changelog.

Changes:

  • Adds API-key authorization and OAuth refresh fix entries.
  • Updates CHANGELOG.md.
File summaries
File Description
CHANGELOG.md Adds authentication-related fixes under Unreleased.
Review details

Suppressed comments (1)

CHANGELOG.md:10

  • This entry is under ### Fixed, but it uses an imperative (Refresh) while the surrounding entries consistently describe completed fixes in the past tense. Please phrase it as a completed fix so the section remains consistent.
- Refresh the OAuth access token before the auth check in generated API methods, instead of only when one was already present
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread CHANGELOG.md
and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).

## [Unreleased]
### Fixed
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