Fix audit log for flat file database #257

Merged
tslocum merged 1 commits from averageByte/tinyib:fix/flatfile-audit-log into master 9 months ago

PHP runs out of memory due to flat file's audit log implementation, and manage mode actions break.

Warning: Undefined array key 1 in /tmp/tinyib/inc/database/flatfile.php on line 207

Warning: Undefined array key 2 in /tmp/tinyib/inc/database/flatfile.php on line 208

Warning: Undefined array key 3 in /tmp/tinyib/inc/database/flatfile.php on line 209

Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 65011720 bytes) in /tmp/tinyib/inc/database/flatfile/flatfile.php on line 344
PHP runs out of memory due to flat file's audit log implementation, and manage mode actions break. ``` Warning: Undefined array key 1 in /tmp/tinyib/inc/database/flatfile.php on line 207 Warning: Undefined array key 2 in /tmp/tinyib/inc/database/flatfile.php on line 208 Warning: Undefined array key 3 in /tmp/tinyib/inc/database/flatfile.php on line 209 Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 65011720 bytes) in /tmp/tinyib/inc/database/flatfile/flatfile.php on line 344 ```
Poster

Closes #245

Closes #245
Owner

Nice catch. Please change $log['id'] to $log[LOG_ID], and then I will merge this. Thanks.

Nice catch. Please change `$log['id']` to `$log[LOG_ID]`, and then I will merge this. Thanks.
Poster

Done.

Done.
tslocum merged commit 648a4fe752 into master 9 months ago
averageByte deleted branch fix/flatfile-audit-log 9 months ago
The pull request has been merged as 648a4fe752.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b averageByte-fix/flatfile-audit-log master
git pull fix/flatfile-audit-log

Step 2:

Merge the changes and update on Forgejo.
git checkout master
git merge --no-ff averageByte-fix/flatfile-audit-log
git push origin master
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: tslocum/tinyib#257
Loading…
There is no content yet.