Skip to content

Remove implicit grant type from samples - #402

Merged
Akanksha Patel (akanpatel2206) merged 2 commits into
release/experimentalfrom
user/akanpatel2206/OAuth2Manager_RemoveImplicit
Jan 15, 2025
Merged

Remove implicit grant type from samples#402
Akanksha Patel (akanpatel2206) merged 2 commits into
release/experimentalfrom
user/akanpatel2206/OAuth2Manager_RemoveImplicit

Conversation

@akanpatel2206

@akanpatel2206 Akanksha Patel (akanpatel2206) commented Jan 15, 2025

Copy link
Copy Markdown

Description

Removing the part from the sample app which uses implicit API for OAuth since that is now removed from OAuth2Manager API as per security concerns.

Target Release

WinAppSDK 1.7 Experimental 2

Checklist

  • Samples build and run using the Visual Studio versions listed in the Windows development docs.
  • Samples build and run on all supported platforms (x64, x86, ARM64) and configurations (Debug, Release).
  • Samples set the minimum supported OS version to Windows 10 version 1809.
  • Samples build clean with no warnings or errors.

Comment thread Samples/OAuth2Manager/cpp-winui/TestOAuthInCPlusPlus/packages.config Outdated
@godlytalias
Godly T.Alias (godlytalias) requested a review from a team January 15, 2025 10:01
@akanpatel2206
Akanksha Patel (akanpatel2206) merged commit d658c33 into release/experimental Jan 15, 2025
@akanpatel2206
Akanksha Patel (akanpatel2206) deleted the user/akanpatel2206/OAuth2Manager_RemoveImplicit branch January 15, 2025 10:27
shisan (qiutongMS) pushed a commit that referenced this pull request Sep 2, 2026
Bring the straightforward release/2.0-stable sample set into main in one reconciliation step: BackgroundTask, CameraCaptureUI, OAuth2Manager, Badge notifications, StoragePickers, Unpackaged WinForms, DrawingIsland, and UXFrameworksOnIslands.

Keep the stable sample behavior while adapting package references, native toolsets, documentation, and CI wiring to the current main infrastructure.

Stable evidence: PRs #367, #392, #395, #398, #400, #402, #403, #405, #413, #421, #422, #427, #455, #512, and #590.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

4 participants