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

add EnableCacheSynchronization #1344

Merged
merged 3 commits into from
Jul 26, 2021
Merged

add EnableCacheSynchronization #1344

merged 3 commits into from
Jul 26, 2021

Conversation

jennyf19
Copy link
Collaborator

@jennyf19 jennyf19 commented Jul 26, 2021

@jennyf19 jennyf19 requested a review from jmprieur July 26, 2021 19:26
@@ -43,6 +43,7 @@ public ConfidentialClientApplicationOptions ConfidentialClientApplicationOptions
public bool IsDefaultPlatformLoggingEnabled { get; set; }
public LogLevel LogLevel { get; set; }
public string? RedirectUri { get; set; }
public bool EnableCacheSynchronization { get; set; }
Copy link
Contributor

@henrik-me henrik-me Jul 26, 2021

Choose a reason for hiding this comment

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

EnableCacheSynchronization

Name is a bit confusing imo, but given that's what we named it in MSAL. #Closed

Copy link
Contributor

@henrik-me henrik-me left a comment

Choose a reason for hiding this comment

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

:shipit:

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@jennyf19 jennyf19 merged commit cea68be into master Jul 26, 2021
@jennyf19 jennyf19 deleted the jennyf/cacheSynchr branch July 26, 2021 20:57
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