Jump to content

My first somewhat fully functional script - what do you think


spyer

Recommended Posts

Good day to you all,

 

well, this is my first script that actually does work on some levels. i do not know if it is OK to post it like what i'll do. i read the rules and could not find the answer..... any-who take a look at it and tell me what you think.

 

Bugs:

it definitely has some bugs in it. do not know what yet, but i'm sure it does.

 

Problems:

it have some problems with language and style selection.

 

NOTE:

  • you might see some Arabic stuff in it that might have slept or caused by an error.. just ignore it :D
  • Admin login information.. USER: admin ... PASS: 123456
  • User login information.. USER: user ... PASS: 123456
  • you'll see one sql file called eu.sql
  • This is the link for the files http://www.easyups.net/eu.zip

 

i really hope that it is in someway good enough!

 

EDIT: sorry, i forgot to mention that it is an uploading script with usercp and admincp.... sorry again :)

Link to comment
Share on other sites

So you want us to test http://www.easyups.net ? You will have a better chance of getting people to test your script if you upload it to a server/host. Unless you mean just use http://www.easyups.net. You never gave a link other then the download one.

 

Most people will not download it. If you decide to upload it to a server/host you should also translate it into English so we know what we're pressing.

Link to comment
Share on other sites

Cross Site Scripting:

You can submit ">code when registering.

 

Full Path Disclosure After Registering:

Warning: mkdir() [function.mkdir]: Permission denied in /mounted-storage/home59c/sub007/sc37417-GQPL/www/eu/register.php on line 119

Warning: mkdir() [function.mkdir]: Permission denied in /mounted-storage/home59c/sub007/sc37417-GQPL/www/eu/register.php on line 125

 

Cross Site Scripting:

http://easyups.net/eu/index.php?styleselect="><marquee><h1>Corey

 

You can just press login on http://easyups.net/eu/admin/index.php and it logs you in.

 

Full Path Disclosure:

http://easyups.net/eu/admin/uploaded.php?do=delete

Warning: fopen(../images/114203/c00aeebe0.jpg) [function.fopen]: failed to open stream: Permission denied in /mounted-storage/home59c/sub007/sc37417-GQPL/www/eu/admin/uploaded.php on line 50

can't open file

Link to comment
Share on other sites

Thank you very much for your time and effort, i really really appreciate it.

 

FIXED

Cross Site Scripting:

You can submit ">code when registering.

 

FIXED

Full Path Disclosure After Registering:

Warning: mkdir() [function.mkdir]: Permission denied in /mounted-storage/home59c/sub007/sc37417-GQPL/www/eu/register.php on line 119

Warning: mkdir() [function.mkdir]: Permission denied in /mounted-storage/home59c/sub007/sc37417-GQPL/www/eu/register.php on line 125

 

FIXED

 

FIXED

You can just press login on http://easyups.net/eu/admin/index.php and it logs you in.

 

FIXED

Full Path Disclosure:

http://easyups.net/eu/admin/uploaded.php?do=delete

Warning: fopen(../images/114203/c00aeebe0.jpg) [function.fopen]: failed to open stream: Permission denied in /mounted-storage/home59c/sub007/sc37417-GQPL/www/eu/admin/uploaded.php on line 50

can't open file

Link to comment
Share on other sites

I've tested it, just a small note, the thumbnail maker does not save the Width/height ratio of the image so of the original image is wider or higher, it will be stretched. stretching not a good idea...

 

there's two ways of fixing this

1- taking the width and height of the original image and divide one over the other to get the width/height ratio, example 800x600 image will have a 800/600 = 1.333 ratio

then you will have a standard width for thumbs, lets say 120, then you will divide the width ( 120 ) with the ratio, so 120/1.333 and you will get the required height 90, so you will need to resize the pic to 120x90... you may want to have another limit for height too, so it's better to make two slightly different codes one for pictures wich are wider ( eg more width than height, like 800x600 ) and another one for pictures wich are higher ( eg more height than width like 600x800 )...

this ay will remain good but will make thumbs size unmatched for each picture. so you want to to go for the second way

 

2- it's same as above, but after resizing it, additional code will add bars to top/button or left/right with certin color ( white or black as example ) in order to keep a specific size of thumbs the you will just copy the thumb over ready made picture that has the exact size as you want...

Link to comment
Share on other sites

thanks xajel,

i've noticed the thumbnails issue, but i had a lot of work these past few days,,, but i'll fix it....

 

 

any other problems or security issues that i should fix :)

 

thanks again for everything....

Link to comment
Share on other sites

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.