php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #61607 geoip.custom_directory gets ignored when set via ini_set
Submitted: 2012-04-03 14:23 UTC Modified: 2014-04-28 18:38 UTC
Votes:7
Avg. Score:4.4 ± 0.5
Reproduced:7 of 7 (100.0%)
Same Version:2 (28.6%)
Same OS:2 (28.6%)
From: max at kriegt dot es Assigned: ohill (profile)
Status: Closed Package: geoip (PECL)
PHP Version: 5.4.0 OS: Debian 6.0.1
Private report: No CVE-ID: None
View Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
If you reported this bug, you can edit this bug over here.
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: max at kriegt dot es
New email:
PHP Version: OS:

 

 [2012-04-03 14:23 UTC] max at kriegt dot es 
Description:
------------
Apache2 via mod_php5
geoip: 1.0.8
geoip lib: 1004007

When setting a custom_directory for geoip, geoip ignores this value until set in 
the php.ini and restarting apache2.

Test script:
---------------
<?php
ini_set('geoip.custom_directory','/tmp/');
$infos = geoip_db_get_all_info();
$info  = array_shift($infos);

var_dump( $info['filename'] );
?>

Expected result:
----------------
string(14) "/tmp/GeoIP.dat"

Actual result:
--------------
string(26) "/usr/share/GeoIP/GeoIP.dat"

Patches

geoip_custom_directory_patch_rev2 (last revision 2012-04-18 01:01 UTC by tiago dot peralta at sapo dot pt)
geoip_custom_directory_patch (last revision 2012-04-18 00:18 UTC by tiago dot peralta at sapo dot pt)

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2012-04-03 20:04 UTC] ohill@php.net
-Status: Open +Status: Not a bug
 [2012-04-03 20:04 UTC] ohill@php.net 
Thank you for taking the time to write to us, but this is not
a bug. Please double-check the documentation available at
http://www.php.net/manual/ and the instructions on how to report
a bug at http://bugs.php.net/how-to-report.php

It is not possible to use ini_set or htaccess.
 [2012-04-18 00:20 UTC] tiago dot peralta at sapo dot pt 
this may not be a bug, but i also found the need to have this feature.

The patch works, the code... well i'm not a C coder so it's really ugly

Best Regards
 [2012-04-18 13:01 UTC] ohill@php.net
-Status: Not a bug +Status: Assigned -Assigned To: +Assigned To: ohill
 [2012-04-18 13:01 UTC] ohill@php.net 
The patch looks good, I'll integrate it in the next release.
 [2013-12-09 15:46 UTC] contact at julienhuang dot com 
It's been more than a year now... it seems that the merge/integration of this patch has been forgotten ... ?
 [2014-04-28 18:35 UTC] ohill@php.net 
The patch is wrong as it would "stick" a change after the request.
 [2014-04-28 18:38 UTC] ohill@php.net 
Automatic comment from SVN on behalf of ohill
Revision: http://svn.php.net/viewvc/?view=revision&amp;revision=333445
Log: Bug #61607
 [2014-04-28 18:38 UTC] ohill@php.net
-Status: Assigned +Status: Closed
 [2014-04-28 18:38 UTC] ohill@php.net 
The fix for this bug has been committed.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.

 For Windows:

http://windows.php.net/snapshots/
 
Thank you for the report, and for helping us make PHP better.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Thu Dec 26 15:01:32 2024 UTC