Modify

Opened 11 years ago

Closed 10 years ago

Last modified 10 years ago

#11205 closed defect (fixed)

Tags of some plugin are broken

Reported by: matobaa Owned by: Michael Renzmann
Priority: normal Component: TracHacks
Severity: normal Keywords:
Cc: Ryan J Ollos, Steffen Hoffmann Trac Release:

Description

Tags of some plugin are broken. It seems be overwritten by spam. TagsPlugin on this site should store changes.

Attachments (0)

Change History (12)

comment:2 Changed 11 years ago by Jun Omae

Cc: Ryan J Ollos Steffen Hoffmann added; anonymous removed

comment:4 Changed 11 years ago by Steffen Hoffmann

For getting back former tags the only hope is to read a db backup from before the SPAM overwrite.

In general tags are currently version-less, and they where ever before. While SPAM is a valid argument for adding versions as demanded by #2429 for years, the design of an appropriate implementation raises some questions.

comment:5 Changed 10 years ago by Steffen Hoffmann

Consider the proposed patch in #2429 for tag revision support to become available soon.

comment:6 Changed 10 years ago by Steffen Hoffmann

In 13428:

TagsPlugin: Introduce tag version recording, requires db upgrade, refs #2429.

This will especially help recovering tags after SPAM tagging, refs #11205.
Therefore configurable behavior enables tracking of wiki page tag changes, but
excludes ticket keywords changes by default.

comment:7 Changed 10 years ago by Ryan J Ollos

Let me know when you feel that the tag versioning added to TagsPlugin is stable, and I'll then go ahead and upgrade TagsPlugin on trac-hacks.

comment:8 in reply to:  3 ; Changed 10 years ago by Ryan J Ollos

Replying to jun66j5:

The following pages have the same issue which is spam tagging and wiki page doesn't exist.

Any idea of the best way to deal with those pages? I'm thinking that we need a script to remove orphaned tags from the database. We should also make sure that tags are deleted when wiki pages are deleted so that this issue can't occur again.

comment:9 in reply to:  8 ; Changed 10 years ago by Steffen Hoffmann

Replying to rjollos:

Replying to jun66j5:

The following pages have the same issue which is spam tagging and wiki page doesn't exist.

Any idea of the best way to deal with those pages? I'm thinking that we need a script to remove orphaned tags from the database.

Yes, but ...

We should also make sure that tags are deleted when wiki pages are deleted so that this issue can't occur again.

This is guaranteed by using wiki page change listener support that has been there since 0.6, so we'll should check an trace of db insanity more closely.

comment:10 in reply to:  8 Changed 10 years ago by Ryan J Ollos

Replying to rjollos:

Replying to jun66j5:

The following pages have the same issue which is spam tagging and wiki page doesn't exist.

Any idea of the best way to deal with those pages?

It was possible to get rid of those tags by creating and then deleting the page.

I looked over the tags page and found a few more spam entries, but it could use another review: tags.

comment:11 in reply to:  9 ; Changed 10 years ago by Ryan J Ollos

Resolution: fixed
Status: newclosed

Replying to hasienda:

We should also make sure that tags are deleted when wiki pages are deleted so that this issue can't occur again.

This is guaranteed by using wiki page change listener support that has been there since 0.6, so we'll should check an trace of db insanity more closely.

It appears that all of the issues in this ticket are resolved, but it would be good to check that tags can't be added to non-existent wiki pages in some way, as you suggest. The issue we were seeing might just be from an earlier implementation of TagsPlugin that wasn't as robust as what we are running now.

The feature in #11661 would be valuable as a next step to help with hunting down spam entries.

comment:12 in reply to:  11 Changed 10 years ago by Steffen Hoffmann

Replying to rjollos:

Replying to hasienda:

We should also make sure that tags are deleted when wiki pages are deleted so that this issue can't occur again.

This is guaranteed by using wiki page change listener support that has been there since 0.6, so we'll should check an trace of db insanity more closely.

It appears that all of the issues in this ticket are resolved, but it would be good to check that tags can't be added to non-existent wiki pages in some way, as you suggest.

I fail to see, when I suggested that, anyway IWikiPageManipulator method [source:tagsplugin/tags/0.7/tractags/wiki.py#L123 WikiTagInterface.validate_wiki_page evaluates page properties that cannot exist for a non-existent Wiki page, so I see no potential issue in the current design.

The issue we were seeing might just be from an earlier implementation of TagsPlugin that wasn't as robust as what we are running now.

Oh, these are/were likely remains of deleted or moved pages before re-parenting (IWikiChangeListener method wiki_page_renamed) and deleting (IWikiChangeListener method wiki_page_deleted) of Wiki page tags had been introduces more recently.

Version 0, edited 10 years ago by Steffen Hoffmann (next)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Michael Renzmann.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.