cnwangjie / phurl

Automatically exported from code.google.com/p/phurl
0 stars 0 forks source link

v3 3 issues #136

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. site.si/admin - not openning and apache error - [Tue May 29 16:32:49 2012] 
[error] [client 85.254.*] PHP Fatal error:  require_once(): Failed opening 
required '../config.php' (include_path='.:/usr/share/php:/usr/share/pear') in 
*/admin/index.php on line 3

2. No idea why this appears - 
[Tue May 29 16:32:54 2012] [error] [client 85.254.*] PHP Warning:  
mysql_real_escape_string(): A link to the server could not be established in 
*/includes/themes/default/header.php on line 45

[Tue May 29 16:32:54 2012] [error] [client 85.254.*] PHP Warning:  
mysql_real_escape_string(): Access denied for user 'www-data'@'localhost' 
(using password: NO) in */includes/themes/default/header.php on line 45

3. and this on shorting urls - [Tue May 29 16:33:46 2012] [error] [client 
85.254.*] PHP Notice:  Undefined index: a in */api/create.php on line 8, 
referer: http://site.si/

btw there are wrong url in homepage when clicking to api - example.org. Must be 
site.si, right?

Original issue reported on code.google.com by edzu...@gmail.com on 29 May 2012 at 1:41

GoogleCodeExporter commented 9 years ago
Thanks for helping out.

1. The admin panel on the repo at the moment is just the one from v2, left 
there for reference. We'll be building a new admin panel, but the priorities 
are more on making the actual shortener work, as we need to do that so we know 
what options etc to include in the admin panel.

2. That's odd. What page are you visiting when that comes up? Just index.php? 

3. Yes, that needs to be fixed. I'll do so shortly.

4. The API link should link to 
http://site.si/api/create.php?url=http://example.org/
Example.org is just there so visitors can understand the API usage with a 
sample URL. I'll probably make a page explaining the API eventually, and link 
it to there then. If the link is actually linking to example.org/api... then 
you need to edit includes/config.php and change the SITE_URL config option.

Hope this helps.

Original comment by he...@phurlproject.org on 29 May 2012 at 1:53

GoogleCodeExporter commented 9 years ago
1. ok, will wait for it. ;)
2. Yes, when opening index.php it gives those 2 errors 2 times - each 2 times.
3. ok
4. the point is that it`s already set - define('SITE_TITLE', 'site.si');

There are <li><a href="/api/create.php?url=http://example.org/">API</a></li>
in includes/themes/default/header.php

Original comment by edzu...@gmail.com on 29 May 2012 at 2:21

GoogleCodeExporter commented 9 years ago
2. That's odd, I'll look into that.

4. It should link to the right place. It's just linking to the API so it 
shortens http://example.org/. Once on the API page, you can change example.org 
to whatever in the address bar and it will shorten the URL you put. Example.org 
is just there to demonstrate how to use the API.

Original comment by he...@phurlproject.org on 29 May 2012 at 2:26

GoogleCodeExporter commented 9 years ago
4. ohh ok, thanks for info.

Original comment by edzu...@gmail.com on 29 May 2012 at 2:56

GoogleCodeExporter commented 9 years ago
2 & 3 are now both fixed in latest commit.

Admin panel will be a while yet I'm afraid :)

Original comment by he...@phurlproject.org on 29 May 2012 at 6:46

GoogleCodeExporter commented 9 years ago
Thanks! Will check tomorrow. By the way - how to do a correct hg update if 
phurl is located in folder called dev ?

And you can let me know if you need to Test if some things working well. ;)

Original comment by edzu...@gmail.com on 29 May 2012 at 7:42

GoogleCodeExporter commented 9 years ago
Probably best to clone it, and keep a clone as an up to date 'blank' copy, then 
copy those files across to wherever you want to test phurl 3. This is what I 
do.. local copy -> test on server -> commit to repo.

Thanks for the offer, I'll let you know if anything crops up :)

Original comment by he...@phurlproject.org on 29 May 2012 at 9:09

GoogleCodeExporter commented 9 years ago
Just updated to new clone. Got php notice when shorting url (writing some 
random url and click shorten) - [Wed May 30 09:48:11 2012] [error] [client 
85.254.*] PHP Notice:  Undefined index: a in */api/create.php on line 8, 
referer: http://site.si/index.php

Original comment by edzu...@gmail.com on 30 May 2012 at 6:52

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
The first error there is something I need to fix - will sort that presently.

The second error is more something I'll have to look into, for the sake of 
avoiding the error. It's basically because now, instead of using mod_rewrite, 
Phurl uses ErrrorDocument 404 in .htaccess to forward URLs. Whilst this works 
great and doesn't require any additional mods, it does log that error (because 
we're essentially tricking it by forwarding 404s to the redirect page which 
then redirects to the target URL). I'll see if there's a way to stop it logging 
the error.

Original comment by he...@phurlproject.org on 30 May 2012 at 9:02

GoogleCodeExporter commented 9 years ago
ok, let me know there if fix comes out ;)

Original comment by edzu...@gmail.com on 30 May 2012 at 9:07

GoogleCodeExporter commented 9 years ago
Fixed in latest commit :)

Original comment by he...@phurlproject.org on 31 May 2012 at 8:07

GoogleCodeExporter commented 9 years ago
still have this issue [Thu May 31 12:37:48 2012] [error] [client 85.254.*] File 
does not exist: */dev/53VAc-

Original comment by edzu...@gmail.com on 31 May 2012 at 9:38

GoogleCodeExporter commented 9 years ago
Yes sorry... I fixed:
[error] [client 85.254.*] PHP Notice:  Undefined index: a in */api/create.php 
on line 8, referer: http://site.si/index.php

The other error is just due to the way Phurl 3 does redirects. It's just 
because /53VAc- does indeed not exist, as it's being redirected to go.php. I 
did some quick research into this but it turns out there's not really a way to 
prevent it. If it really bugs you, I'm sure there's a way you can configure 
your PHP installation to ignore it, but script side, there is nothing I can do.

Original comment by he...@phurlproject.org on 31 May 2012 at 9:42

GoogleCodeExporter commented 9 years ago
i`m wondering why /53VAc (without - ) not giving the same error? 

Original comment by edzu...@gmail.com on 31 May 2012 at 9:48

GoogleCodeExporter commented 9 years ago
Try shortening another URL and visiting that.. you should see the error. 
I think it's because your browser has cached the redirect so it's not actually 
hitting the page.

Original comment by he...@phurlproject.org on 31 May 2012 at 9:56