causefx / iDashboard-PHP

HTPC Dashboard to load website services, written in PHP
MIT License
123 stars 20 forks source link

Error on settings #3

Closed mmolitor87 closed 8 years ago

mmolitor87 commented 8 years ago

Ubuntu 14.04.4 LTS Apache/2.4.7 PHP 5.5.9-1ubuntu4.16 (cli)

Here's what I ran: git clone https://github.com/causefx/iDashboard-PHP test

chown www-data:www-data test -R

Login and go to settings page to receive:

$section) { if(($keyname == "general")) { $nopass = $section["password"]; } } $salt = substr(str_replace('+','.',base64_encode(md5(mt_rand(), true))),0,16); $rounds = 10000; if(empty($_POST["password-0"])){ $_POST["password-0"] = $nopass;} if(strlen($_POST["password-0"]) < 50){ $_POST["password-0"] = crypt($_POST["password-0"], sprintf('$5$rounds=%d$%s$', $rounds, $salt)); } //password crap $sampleData .= '; Access denied

causefx commented 8 years ago

I actually haven't tried cloning. Let me try now.

causefx commented 8 years ago

@mmolitor87 can you try this: https://github.com/causefx/iDashboard-PHP/releases/tag/0.97

hjone72 commented 8 years ago

Issue is still exists. Does it matter that I am running this at /dashboard ?

causefx commented 8 years ago

It doesn't matter where u run it. I'll keep looking into it.

mmolitor87 commented 8 years ago

@causefx:

git checkout 851c6ba Note: checking out '851c6ba'.

Same error happened. :/

Also wanted to let you know that it is happening when I extract a zip as well. So it isn't just a cloning issue.

More fun:

git checkout d27c201 = Infinite reloading issue in the other thread. Means it isn't contained to php 7.0 like you thought in the other thread.

git checkout d27c201 = Only tabs show.

git checkout 9b87b4f = Release I started on, so this issue.

causefx commented 8 years ago

@mmolitor87 Thanks for the write up. Good info. Shit's got me scratching my had because it throws no errors.

mmolitor87 commented 8 years ago

@causefx I work in IT helpdesk so I have a good idea of what info is helpful!

It looks to me as if the code isn't parsing properly and is just outputting part of the login code (password handling) right to the page. I don't know enough about php to troubleshoot the code itself, though.

causefx commented 8 years ago

You are correct but for some reason it's also reloaded the page, correct?

mmolitor87 commented 8 years ago

It is NOT reloading the page with this error. It is a constant error shown with no loading animation.

causefx commented 8 years ago

@mmolitor87 can you try this?

https://github.com/causefx/iDashboard-PHP/archive/0.97-2.zip

mmolitor87 commented 8 years ago

@causefx Well it loads.

bug

causefx commented 8 years ago

LOL, half way there.

causefx commented 8 years ago

@mmolitor87 can you do me a favor...?

can you get me the frame source on that tab? also, can you get me the contents of settings.ini.php?

mmolitor87 commented 8 years ago

@causefx

settings.ini.php.txt

source.txt

Not sure if this is the source you're looking for.

causefx commented 8 years ago

@mmolitor87 that source is from the main page, we need it from the frame...

image

Also, does your settings.ini.php not have line breaks?
the file you sent looked like this: image

Should look like this: image

mmolitor87 commented 8 years ago

@causefx

Here is the frame source: source.txt

Open the settings with notepad++ - for some reason the line breaks aren't preserved otherwise.

causefx commented 8 years ago

ok, let me do one fix...

causefx commented 8 years ago

@mmolitor87 try this:

https://github.com/causefx/iDashboard-PHP/archive/0.97-3.zip

we should have some progress on this...

mmolitor87 commented 8 years ago

@causefx Some progress:

bug

causefx commented 8 years ago

any chance on getting the source again :)

mmolitor87 commented 8 years ago

source.txt

causefx commented 8 years ago

dude, you're awesome man! thanks for helping me... shits kicking my ass :(

for some reason it isnt switching back to php after the "<?"

mmolitor87 commented 8 years ago

You're welcome!

I don't know if it matters but line 94 is ' <?$i = 0;'

and 110-112:

            <?}
            $i++;
        }?>

should the php brackets be separate from other bits of code?

causefx commented 8 years ago

@mmolitor87 I have 2 things left to try....

https://github.com/causefx/iDashboard-PHP/archive/0.97-4.zip

mmolitor87 commented 8 years ago

@causefx No change with that commit on the settings tab. Here's the source just in case:

source.txt

causefx commented 8 years ago

need the frame source :(

mmolitor87 commented 8 years ago

D'oh I'm a dummy.

source.txt

causefx commented 8 years ago

weird, the changes didnt even come thru

causefx commented 8 years ago

@mmolitor87 can you refresh your browser?

the code didnt update when you uploaded for some reason....

image

should look like that

mmolitor87 commented 8 years ago

yay

That's because I didn't pull the commit properly. ashamed

causefx commented 8 years ago

@mmolitor87

holy shit!!!! it was a simple fix on my end... <? -> <?php which shouldnt matter.... so you're all good now?

mmolitor87 commented 8 years ago

Yup! Sure am. :) Now for that color picker for all my troubleshooting assistance. ;)

causefx commented 8 years ago

hahaha yes! for sure :)

how do you want it the color changes? just for the tabs? different color for active tab? border? background?

mmolitor87 commented 8 years ago

The tabs/settings page. Since they act as a container those are the only places they would change, really.

My only other idea for you would be a way to individually refresh tabs. :) I don't want to overload you, though.

causefx commented 8 years ago

actually on that last commit I added code to start the refresh of tab... i hope soon to have that done.

causefx commented 8 years ago

Resolved in https://github.com/causefx/iDashboard-PHP/archive/0.98.zip