Skip to content
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

Extract mid and pid from GET Parameters instead of REQUEST #356

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cs8898
Copy link

@cs8898 cs8898 commented Jul 13, 2023

Sometimes You will get an Invalid Tab error in the webui, its not possible to access any pages except the Dashboard.

It turned out, that in the cookies there is also an key m that will overwrite the $_GET['m'].
Because the Tabs should always be send by GET i just replaced the origin of the GrabGoodInput.

https://forum.openlitespeed.org/threads/invalid-tabs-2258.5515/
https://community.cyberpanel.net/t/openlitespeed-invalid-tabs-error/41101

@cs8898 cs8898 changed the title Extract mid and pid from GET Parameters instead of REQUEST WIP: Extract mid and pid from GET Parameters instead of REQUEST Jul 13, 2023
@cs8898
Copy link
Author

cs8898 commented Jul 13, 2023

Noticed, that tab views sometimes use POST (eg. Add VHOST)

@cs8898 cs8898 changed the title WIP: Extract mid and pid from GET Parameters instead of REQUEST Extract mid and pid from GET Parameters instead of REQUEST Jul 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant