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

node/index.php attempts to send entire log on visit #325

Closed
LordRalex opened this issue Dec 7, 2014 · 3 comments
Closed

node/index.php attempts to send entire log on visit #325

LordRalex opened this issue Dec 7, 2014 · 3 comments
Labels
Milestone

Comments

@LordRalex
Copy link
Member

If your server is running, when you visit /node/index.php, Unirest will send the entire log file. If your server is in general blowing up or has a large log, then PHP will try to allocate all that memory, and in instances, will die because of php.ini limits. There should be some limit to this (within a minute i had a 60MB log due to GSD errors)

This is probably related to GSD being the issue because of the unirest call, but it is an issue on the site because of it.

Related line to this: https://github.com/PufferPanel/PufferPanel/blob/master/panel/node/index.php#L31

@DaneEveritt
Copy link
Contributor

This is a double program issue. What we really need for this is a GSD function that returns the last x bytes of a file. This is the same issue that occurs when trying to download a file, so I think I might move this to a FTP fix since I believe I can open a stream and read the last x bytes much easier through that.

Referencing #315

@DaneEveritt DaneEveritt modified the milestone: 0.8.0 Beta Dec 7, 2014
@DaneEveritt
Copy link
Contributor

Looks like using the function below might allow for this to occur from the panel. We would need to open a FTP connection through rather than using GSD.

stream_get_line();

@DaneEveritt DaneEveritt modified the milestones: 0.8.0 Beta, 0.7.6 Beta Dec 7, 2014
@LordRalex
Copy link
Member Author

Having GSD handle this seems to be a better approach than to work on a different solution through the panel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants