We have moved to Github. Please open tickets there.

Opened 13 months ago

Closed 11 months ago

Last modified 11 months ago

#4487 closed Defect (fixed)

system_log(): Sniff the version and only use ip_address for latest version.

Reported by: brettp Owned by:
Priority: critical Milestone: Elgg 1.8.6
Component: Core Version: 1.8
Severity: blocker Keywords:
Cc: brett@… Difficulty: easy

Description

Problematic upgrades if going from < 1.8.3 to 1.8.4.

Change History (8)

comment:1 Changed 13 months ago by brettp

Got to thinking, instead of a version sniff how does everyone feel if I add an internal var that will disable the sys log, and then we use that during upgrades? I don't think the syslog is particularly useful for upgrades.

comment:2 Changed 12 months ago by brettp

  • Resolution set to duplicate
  • Status changed from new to closed

Dupe of #4326. Now to figure out why it's not working correctly.

comment:3 Changed 12 months ago by Brett Profitt

Refs #4487. Turning off system log (even more) during upgrades. Removed debug call to system_log() in create_annotation().

Changeset: 25b1a1ea11182b6fc195f4cca76ec4788d79c65e

comment:4 Changed 12 months ago by brettp

  • Difficulty changed from trivial to easy
  • Resolution duplicate deleted
  • Status changed from closed to reopened

comment:5 Changed 12 months ago by captnbob

Brett, you asked for a Stack Trace on this problem. Here is mine with the exception being that this is a new install. I added a plugin and ran "upgrade.php" and got the same error:
Stack trace:
#0 /home/mysite/engine/lib/database.php(493): execute_query()
#1 /home/mysite/engine/lib/upgrades/2012012000-1.8.3-ip_in_syslog-87fe0f068cf62428.php(12): update_data()
#2 /home/mysite/engine/lib/upgrade.php(62): include('/home/mysite/...')
#3 /home/mysite/engine/lib/upgrade.php(255): upgrade_code()
#4 /home/mysite/upgrade.php(27): version_upgrade()
#5 {main}
[code:protected] => 0
[file:protected] => /home/mysite/engine/lib/database.php
[line:protected] => 261
[trace:private] => Array
(
[0] => Array
(
[file] => /home/mysite/engine/lib/database.php
[line] => 493
[function] => execute_query
)

[1] => Array
(
[file] => /home/mysite/engine/lib/upgrades/2012012000-1.8.3-ip_in_syslog-87fe0f068cf62428.php
[line] => 12
[function] => update_data
)

[2] => Array
(
[file] => /home/mysite/engine/lib/upgrade.php
[line] => 62
[args] => Array
(
[0] => /home/mysite/engine/lib/upgrades/2012012000-1.8.3-ip_in_syslog-87fe0f068cf62428.php
)

[function] => include
)

[3] => Array
(
[file] => /home/mysite/engine/lib/upgrade.php
[line] => 255
[function] => upgrade_code
)

[4] => Array
(
[file] => /home/mysite/upgrade.php
[line] => 27
[function] => version_upgrade
)

)

)

comment:6 Changed 12 months ago by Brett Profitt

Refs #4487. Turning off system log (even more) during upgrades. Removed debug call to system_log() in create_annotation().

Changeset: 25b1a1ea11182b6fc195f4cca76ec4788d79c65e

comment:7 Changed 11 months ago by Cash Costello

  • Resolution set to fixed
  • Status changed from reopened to closed

Fixes #4487 turned off logging for sites that haven't been upgraded with ip address column

Changeset: 242ea5fa2b1ac775b74cf118a8b81d79e531104a

comment:8 Changed 11 months ago by Cash Costello

Fixes #4487 turned off logging for sites that haven't been upgraded with ip address column

Changeset: 242ea5fa2b1ac775b74cf118a8b81d79e531104a

Note: See TracTickets for help on using tickets.