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

assign post_max_size to fix https://github.com/usmannasir/cyberpanel/issues/1163 #1164

Closed
wants to merge 5 commits into from
Closed

Conversation

Lvl4Sword
Copy link
Contributor

I think this is all that's needed here, as the error is pretty straightforward.

with open() is now used to open files and properly close them.
proper indentation is certain places.
Some repeat code is commented as being as such.
@Lvl4Sword
Copy link
Contributor Author

Lvl4Sword commented Dec 2, 2023

Should Django's JsonResponse be used in fetchPHPExtensions rather than the HttpResponse used here?

@Lvl4Sword
Copy link
Contributor Author

listChildDomainsJson was added and will fix #1120
Also did args.function.lower() so cli commands don't need specific casing.

@Lvl4Sword
Copy link
Contributor Author

Weirdly enough listChildDomainsJson brings back PHP 7.3 for php_version. Are child domains setting PHP incorrectly?

@Lvl4Sword
Copy link
Contributor Author

@usmannasir Can you take a look at this?

@usmannasir
Copy link
Owner

I am sorry I was out of the country. Right now I can see that only the cli file is changed?

@GoldenSwordLLC GoldenSwordLLC closed this by deleting the head repository Jul 19, 2024
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.

3 participants