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

VerifyDBAbstractionLib() Not operative under safe mode. #34

Closed
NathanGibbs3 opened this issue May 30, 2019 · 3 comments
Closed

VerifyDBAbstractionLib() Not operative under safe mode. #34

NathanGibbs3 opened this issue May 30, 2019 · 3 comments
Assignees
Labels
DB Database Issues invalid This doesn't seem right LCB-TechDebt Issue exists in Legacy Code Base. We inherited it. Prod Observed in Production Environment. Stability Issue impacts or is related to App Stability.
Milestone

Comments

@NathanGibbs3
Copy link
Owner

NathanGibbs3 commented May 30, 2019

Item Description
File: includes/base_db.inc.php
Class: N/A
Function: VerifyDBAbstractionLib()
Similar Issues:
Related Issues: https://sourceforge.net/p/secureideas/bugs/47
Depends on Issue(s):
Dependency Type: N/A
Misc. Info.: The safe mode cutout was added on 20050325. It may only be needed on PHP Versions below 5.1.5 as PHP 5.1x was released on 20051124. See https://www.php.net/manual/en/function.is-readable.php Will need to research this further.

Expected Behavior: VerifyDBAbstractionLib() is operative under safe mode where possible.
Current Behavior: VerifyDBAbstractionLib() function is not operative under safe mode.

@NathanGibbs3 NathanGibbs3 added invalid This doesn't seem right question Further information is requested Prod Observed in Production Environment. LCB-TechDebt Issue exists in Legacy Code Base. We inherited it. labels May 30, 2019
@NathanGibbs3
Copy link
Owner Author

NathanGibbs3 commented May 31, 2019

Research Results: PHP Safe Mode History.
Safe Mode has been around since before Version 4.0, Beta 4
See: https://www.php.net/ChangeLog-4.php
Was deprecated in PHP 5.3x and removed in PHP 5.4x.
See: https://www.php.net/manual/en/features.safe-mode.php

@NathanGibbs3 NathanGibbs3 removed the question Further information is requested label May 31, 2019
NathanGibbs3 added a commit that referenced this issue May 31, 2019
NathanGibbs3 added a commit that referenced this issue May 31, 2019
NathanGibbs3 added a commit that referenced this issue May 31, 2019
NathanGibbs3 added a commit that referenced this issue May 31, 2019
NathanGibbs3 added a commit that referenced this issue May 31, 2019
NathanGibbs3 added a commit that referenced this issue May 31, 2019
NathanGibbs3 added a commit that referenced this issue May 31, 2019
@NathanGibbs3
Copy link
Owner Author

NathanGibbs3 commented May 31, 2019

Verified: Works with PHP Safe Mode enabled on PHP 5.3.29 via CI & Unit Tests..
See: https://travis-ci.org/NathanGibbs3/BASE/jobs/539843474
See: https://travis-ci.org/NathanGibbs3/BASE/job**s/539843475
Conclusion: Should work with PHP Safe Mod enabled in PHP > 5.1.4.
See https://www.php.net/manual/en/function.is-readable.php

NathanGibbs3 added a commit that referenced this issue May 31, 2019
NathanGibbs3 added a commit that referenced this issue Jun 1, 2019
NathanGibbs3 added a commit that referenced this issue Jun 2, 2019
NathanGibbs3 added a commit that referenced this issue Jun 2, 2019
NathanGibbs3 added a commit that referenced this issue Jun 3, 2019
NathanGibbs3 added a commit that referenced this issue Jun 5, 2019
NathanGibbs3 added a commit that referenced this issue Jun 5, 2019
         Cleanup Unit Test fallout from fixes for #34 & #35.
         Cleaned up code in tests/php/langtdflegacyTest.php
NathanGibbs3 added a commit that referenced this issue Jun 5, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 5, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 5, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 5, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 5, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 5, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 5, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 5, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 5, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 5, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 5, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 5, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 6, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 6, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 6, 2019
         Cleanup Unit Test fallout from fixes for #34.
@NathanGibbs3
Copy link
Owner Author

When we merged this fix into the Issue #11 branch, Unit Tests that needed process isolation started causing the builds to fail on PHP 5.3x due to safe mode deprecation errors.

NathanGibbs3 added a commit that referenced this issue Jun 6, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 6, 2019
         Cleanup Unit Test fallout from fixes for #34.
NathanGibbs3 added a commit that referenced this issue Jun 6, 2019
         Reverted the Issue #34 related trainwreck on this branch.
         Back to commit: a688054
         Keep the Issue #35 corrections.
         Will fix the underlying Issue #34 problem from the
         devel branch and then remerge.
@NathanGibbs3 NathanGibbs3 self-assigned this Jun 26, 2019
@NathanGibbs3 NathanGibbs3 added the DB Database Issues label May 21, 2020
@NathanGibbs3 NathanGibbs3 added this to the 1.4.6 milestone Oct 8, 2020
@NathanGibbs3 NathanGibbs3 added the Stability Issue impacts or is related to App Stability. label Dec 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DB Database Issues invalid This doesn't seem right LCB-TechDebt Issue exists in Legacy Code Base. We inherited it. Prod Observed in Production Environment. Stability Issue impacts or is related to App Stability.
Projects
None yet
Development

No branches or pull requests

1 participant