The Wayback Machine - https://web.archive.org/web/20201221014044/https://github.com/beyondcode/expose/pull/9
Skip to content

/ expose

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

Jump to bottom

[WIP] Pre-release changes/fixes #9

Merged
merged 5 commits into from Jun 15, 2020
Merged

[WIP] Pre-release changes/fixes #9

merged 5 commits into from Jun 15, 2020

Conversation

@raulp
Copy link
Collaborator

@raulp raulp commented Jun 14, 2020

  1. Fixes some memory issues due to large responses and the way the responses are send to the local dashboard. In current version they are kept in memory, and on each new request, they last 25 (or X configurable requests) are re-looped and the body is re-read again. This will cause a memory issue when refreshing a page 3-5 times, depending on how heavy the resources on that page are. I changed the way the requests are sent from RequestLogger to the local dashboard, by only sending 1 at the time, as we get them. Then, the dashboard socket interface was changed to listen for 1 request only.
  2. Added dashboard title
  3. Added JSON syntax highlighter for body previews
  4. added config variable for memory_limit, defaults to 128M
  5. added config variable for things that should skip the body log (skip_body_log - TBD)
  6. removed clear logs: as all processing of the logs is done in the interface, this is redundant.
  7. updated composer dependencies. That's needed because of a PR merged in reactphp that fixed closing sockets before full content was passed.
raulp added 5 commits Jun 14, 2020
@raulp
pre-release changes
23c4543
@raulp
update dependecies - needed for mac
6100d9d
@raulp
fix merge from old repo
4e7db29
@raulp
fix merge from old repo
c53eb3c
@raulp
build expose
1b5aa57
@raulp raulp changed the title Pre-release changes/fixes Jun 14, 2020
@mpociot mpociot merged commit 0b76358 into beyondcode:master Jun 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
@raulp @mpociot
You can’t perform that action at this time.