-
Notifications
You must be signed in to change notification settings - Fork 22
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
Too many redirects #47
Comments
From @mortezakermani there are a lot of ?portal=0 parameters in url and site breaks The last version of Active Directory worked for me is 05.00.02. no other versions work correctly. |
I have seen this in production rarely. When I do see it, a refresh always makes it go away. I need as many details as you can possible supply to track down the issue. I will do my best to help! |
I think I have it already narrowed down. I suspect that its the auto-login ip address. I am refactoring to allow more situations. Can you try adding an IP range in the autologin ip field that would include the client IP you are using? Please report your results. If your client IS the web server also (localhost) then there is a Microsoft problem related to that I will address later. But please report if this is the situation also. |
I tried both "Auto-login" and no "Auto-login". Same result. Just my 2 cents. Off topic, can someone share v5.0.0.2 to me? |
I install v5.0.0.6 .
|
I have an update coming soon that should get this fixed. It appears the redirect problem will arise if you do not have an IP in the auto-login box. I you add a range such as 1.1.1.1-255.255.255.255 then the redirect problem should go away. |
We add a checking, if "portalid=0" exists in query string, then do not redirect. |
Thanks a lot @ghkamfung. may you release a new version in this week? |
I should have a new version out this week with the fix. Its complete I just need to clean up the code some. @ghkamfung its proper for the url to contain 1 portalid=0 and this will break again on a different portal of the same DNN instance as its portal id will be something other than 0. It should get you by though until I can get this update out. |
@ghkamfund and @mortezakermani Please try the new release 6.1.2 as it should have this bug fixed. Be sure to utilize the new Enable Auto Login setting feature. |
v6.1.2 vIt works for us! Thank you very much. |
Excellent. Thanks for the feed back. |
From @ghkamfung
Something like this:
http://10.145.2.77/?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0?portalid=0
The text was updated successfully, but these errors were encountered: