Last modified: 2014-08-26 21:30:14 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T71049, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 69049 - All the revision IDs of a page should be stored in log_params when that page is deleted
All the revision IDs of a page should be stored in log_params when that page ...
Status: PATCH_TO_REVIEW
Product: MediaWiki
Classification: Unclassified
Page deletion (Other open bugs)
1.24rc
All All
: Lowest enhancement (vote)
: ---
Assigned To: Nathan Larson
:
Depends on:
Blocks: 69047
  Show dependency treegraph
 
Reported: 2014-08-02 14:39 UTC by Nathan Larson
Modified: 2014-08-26 21:30 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Nathan Larson 2014-08-02 14:39:43 UTC
All the revision IDs of a page should be stored in log_search when that page is deleted. That way, if the page is undeleted, there is still a way to find out what page ID its revisions had at the time of deletion. This is useful for countering the merge page histories attack; see bug 69047.
Comment 1 Gerrit Notification Bot 2014-08-02 17:45:22 UTC
Change 151309 had a related patch set uploaded by leucosticte:
Store revision IDs of a page in log_search when that page is deleted

https://gerrit.wikimedia.org/r/151309
Comment 2 Nathan Larson 2014-08-02 17:50:06 UTC
Hey MZM, do you think there might be a more efficient way to accomplish what this patch does? As written, the patch does a select on the revision table on the master to get the revision IDs right before an insertSelect is done on archive, page, and revision on the master to shunt the revision data into the archive table. Is there a way to add this functionality without adding another query, or is there a way to do it using a less expensive query? Thanks.
Comment 3 Nathan Larson 2014-08-23 00:54:09 UTC
So is this a WONTFIX? The log_search option was opposed by Brian Wolff and the log_params option was described on IRC by legoktm as infeasible.
Comment 4 Nathan Larson 2014-08-26 21:30:14 UTC
Another possible implementation would be to have page_original_id and ar_original_page_id fields that contain the value of what the page_id was when the page was created. Revisions could then be merged or unmerged based on the value of that field.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links