Dnn.Platform icon indicating copy to clipboard operation
Dnn.Platform copied to clipboard

[Bug]: SimpleWebFarmCachingProvider Messages Captured by Friendly URL Provider

Open mitchelsellers opened this issue 2 years ago • 3 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

What happened?

When using the SimpleWebFarmCachingProvider the DNN Friendly URL system is capturing the requests and returning them as a 404.

This means that the caching provider is not working, 404's are logged, and site performance is poor.

To work around this you have to update the URL Exclusions Regex to add this element |/SimpleWebFarmSync\.aspx

Steps to reproduce?

  1. Enable the Simple Web Farm Caching Provider
  2. Look at the log entries and see 404's

Current Behavior

Any request to sync cache fails with a 404

Expected Behavior

Cache sync requests should work as expected

Relevant log output

No response

Anything else?

Not sure how we want to handle this, as not all sites need the modification, but we need to ensure that users know that they need to add this value.

Affected Versions

9.12.0 (latest release)

What browsers are you seeing the problem on?

Firefox, Chrome, Safari, Microsoft Edge

Code of Conduct

  • [X] I agree to follow this project's Code of Conduct

mitchelsellers avatar Jun 23 '23 21:06 mitchelsellers

That exclusion would not affects users that don't use that feature right? So we could just make it so.

valadas avatar Jun 23 '23 22:06 valadas

Can the provider check for that exclusion and add it if it's missing (on app start, when it's enabled)? I'm also okay with adding it all of the time, I guess we'd just need to add some upgrade logic.

bdukes avatar Jun 26 '23 13:06 bdukes

Can the provider check for that exclusion and add it if it's missing (on app start, when it's enabled)

This is an interesting idea, I assume setting it properly and clearing the cache would allow it to be captured/effective right away.

My only fear here is custom edits of that field? Which is an issue on both sides

mitchelsellers avatar Jul 17 '23 03:07 mitchelsellers