Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: [.NET Core] Remove .UseAuthorization if unnecessary for running bots #2920 #3978

Merged
merged 5 commits into from
Jun 3, 2024

Conversation

Amrit10737726
Copy link
Contributor

@Amrit10737726 Amrit10737726 commented May 22, 2024

#minor
#2920

Description
Removing .UseAuthorization() in Startup.cs from dotnetcore samples because it is not required. All authorization and authentication occurs within the SDK.

…ed in the bot sample and is also not used by the SDK.
…ed in the bot sample and is also not used by the SDK.
@Amrit10737726 Amrit10737726 self-assigned this May 22, 2024
@Amrit10737726 Amrit10737726 requested a review from a team as a code owner May 22, 2024 18:01
@Amrit10737726 Amrit10737726 changed the title V amrit10737726/removeauthorization fix:[.NET Core] Remove .UseAuthorization if unnecessary for running bots #2920 May 22, 2024
@Amrit10737726 Amrit10737726 changed the title fix:[.NET Core] Remove .UseAuthorization if unnecessary for running bots #2920 fix: [.NET Core] Remove .UseAuthorization if unnecessary for running bots #2920 May 22, 2024
…thCertificateBot.csproj

removing due to accidentally added
@tracyboehrer tracyboehrer merged commit ce328b0 into main Jun 3, 2024
31 of 32 checks passed
@tracyboehrer tracyboehrer deleted the v-amrit10737726/removeauthorization branch June 3, 2024 17:28
@gandiddi gandiddi linked an issue Jun 3, 2024 that may be closed by this pull request
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.

[.NET Core] Remove .UseAuthorization if unnecessary for running bots
3 participants