-
Notifications
You must be signed in to change notification settings - Fork 62
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
Custom List Icon Bad URL causes site to logout after 5 seconds #2512
Comments
There are at least 2 approaches to fixing this. First, put some verification that the URL is pointing to an accessible resource when the item is saved, but if the item is deleted later we could still have the same problem. Option 2 is to figure out why the user is logged out when it hits a bad resource like this. |
Check for new notifications every 5 mins, if that fails (auth expired) then reload.
The auto sign out issue i think was related to something else. I'll try to find it. |
Still Todo: Option 2 is to figure out why the user is logged out when it hits a bad resource like this. |
Describe the bug
If the URL for a custom list item icon points to a resource that doesn't exist within the DT server, the response will come back as a 302 error. That for some reason clears the logged in cookie, so the later API calls, in my case the
get_new_notifications_count
call will fail and the user will be logged out. This will happen on every page that the icon with the faulty URL appears.To Reproduce
Steps to reproduce the behavior:
/wp-content/images/
instead ofwp-content/themes/disciple-tools-theme-master/dt-assets/images/
The only way I could fix this to go into the DB into the options table and edit the value for
dt_field_customizations
to change the URL.The text was updated successfully, but these errors were encountered: