You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
hi i liked your app.. revised the code.. You have a fairly abstract vision, which makes it difficult to collaborate if you don't have good programming skills specially when dont have good OOP skills ... that makes it difficult for any person to collaborate in your application
currently the htpasswd class ( at tools/htpasswd.php) only handles a single file and assumes a single path , so it retrieves the path from the protected directory, that's fine only for shitty apache like noob admins..
in best practice deploys those files are far away from that dir,, so i guess i better to change it on the ini file in that way:
make a new key item named secure_file or htpasswd file
in constructor of htpasswd class ( at tools/htpasswd.php) add the supprot of independient paths
make a first string extract with $fch = substr($htpasswdfile, 0, 1);
compared if start with / suing strcmp($fch, '/') if TRUE then use absolute paths,
make a second string extract with $fch = substr($htpasswdfile, 0, 3);
compared if start with ../ suing strcmp($fch, '../') if TRUE use directory app path as reference
if both fails.. use the protected directory
thanks in advance, i suggest another more advanced option.. by usage the more advanced htpasswd class that support groups
hi i liked your app.. revised the code.. You have a fairly abstract vision, which makes it difficult to collaborate if you don't have good programming skills specially when dont have good OOP skills ... that makes it difficult for any person to collaborate in your application
currently the
htpasswd
class ( attools/htpasswd.php
) only handles a single file and assumes a single path , so it retrieves the path from the protected directory, that's fine only for shitty apache like noob admins..in best practice deploys those files are far away from that dir,, so i guess i better to change it on the ini file in that way:
secure_file
orhtpasswd
filehtpasswd
class ( attools/htpasswd.php
) add the supprot of independient paths$fch = substr($htpasswdfile, 0, 1);
/
suingstrcmp($fch, '/')
ifTRUE
then use absolute paths,$fch = substr($htpasswdfile, 0, 3);
../
suingstrcmp($fch, '../')
ifTRUE
use directory app path as referencethanks in advance, i suggest another more advanced option.. by usage the more advanced htpasswd class that support groups
that class provides a htpasswd with group support!
The text was updated successfully, but these errors were encountered: