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.

Sign up for GitHub

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

Enable resetting application context user to null #4415

Merged
merged 10 commits into from
Dec 30, 2024
Merged

Enable resetting application context user to null #4415

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
6512faf
Set application context user to an unauthenticated claims principal w…
StefanOssendorf Dec 23, 2024
8eeb1da
Remove null-check in ApplicationContextManagerHttpContext
StefanOssendorf Dec 24, 2024
d2ab58b
Merge branch 'main' into dev/fix-4410-flowsecurityprinciaplfromclient
StefanOssendorf Dec 24, 2024
b07a0ab
Merge branch 'main' into dev/fix-4410-flowsecurityprinciaplfromclient
rockfordlhotka Dec 30, 2024
1019f8b
Add test to test for not null cleanup in DataPortal
StefanOssendorf Dec 30, 2024
62287a5
Make the change less impactful to users
StefanOssendorf Dec 30, 2024
55273db
Merge branch 'main' into dev/fix-4410-flowsecurityprinciaplfromclient
StefanOssendorf Dec 30, 2024
db05e6b
Unused decorater removed
StefanOssendorf Dec 30, 2024
5b54d03
Update migration doc
StefanOssendorf Dec 30, 2024
c479d2d
Fix CheckInnerExceptionsOnSave test
rockfordlhotka Dec 30, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion Source/Csla/Server/DataPortal.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
//-----------------------------------------------------------------------

using System.Diagnostics.CodeAnalysis;
using System.Security.Claims;
using System.Security.Principal;
using Csla.Configuration;
using Csla.Properties;
Expand Down Expand Up @@ -774,7 +775,7 @@ private void ClearContext(DataPortalContext context)
if (!context.IsRemotePortal) return;
_applicationContext.Clear();
if (SecurityOptions.FlowSecurityPrincipalFromClient)
_applicationContext.User = null;
_applicationContext.User = new ClaimsPrincipal(new ClaimsIdentity());
rockfordlhotka marked this conversation as resolved.
Show resolved Hide resolved
}

#endregion
Expand Down