Skip to content

Fixed Cookie Timeout Lookup Values#6814

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

Fixed Cookie Timeout Lookup Values#6814
bdukes merged 1 commit intodnnsoftware: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
Copy link
Copy Markdown
Contributor

@valadas valadas left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@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