Skip to content

Fixed Cookie Timeout Lookup Values#6814

Merged
bdukes merged 1 commit into
dnnsoftware:developfrom
mitchelsellers:feature/fix-cookie-persistence-6811
Dec 2, 2025
Merged

Fixed Cookie Timeout Lookup Values#6814
bdukes merged 1 commit into
dnnsoftware:developfrom
mitchelsellers:feature/fix-cookie-persistence-6811

Conversation

@mitchelsellers

Copy link
Copy Markdown
Contributor

Summary

The existing behavior blindly assumed that if a location node exists that the system.web.authencation/forms element is in the location node.

This updates the behavior to start with the normal location, IF if funds forms authenticaton it continues, otherwise, it tries to find it under a location node IF location nodes exist.

This should restore functionality after the updates recently resulted in ALL installations having a location node. This has been tested and appears to be working well

@mitchelsellers mitchelsellers added this to the 10.1.3 milestone Nov 25, 2025

@valadas valadas 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.

Looks good, thanks!

Comment thread DNN Platform/Library/Common/Utilities/Config.cs Outdated
@bdukes
bdukes force-pushed the feature/fix-cookie-persistence-6811 branch from 162499f to 63a8c2b Compare December 2, 2025 20:44
The existing behavior blindly assumed that if a location node exists that the system.web.authencation/forms element is in the location node.

This updates the behavior to start with the normal location, IF if funds forms authenticaton it continues, otherwise, it tries to find it under a location node IF location nodes exist.

This should restore functionality after the updates recently resulted in ALL installations having a location node.  This has been tested and appears to be working well
@bdukes
bdukes force-pushed the feature/fix-cookie-persistence-6811 branch from 63a8c2b to 74424c3 Compare December 2, 2025 20:45
@bdukes
bdukes enabled auto-merge (rebase) December 2, 2025 21:00
@bdukes
bdukes merged commit 464f908 into dnnsoftware:develop Dec 2, 2025
3 checks passed
@valadas valadas modified the milestones: 10.1.3, 10.2.0 Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Cookie persistent timeout not enforced on DNN 10.1.2

4 participants