Closed (fixed)
Project:
Show unpublished comments
Version:
6.x-1.2
Component:
Code
Priority:
Minor
Category:
Support request
Assigned:
Reporter:
Created:
3 Feb 2010 at 22:24 UTC
Updated:
13 Sep 2010 at 11:20 UTC
Jump to comment: Most recent file
Comments
Comment #1
siliconmind commentedI wasn't able to reproduce this issue. Can you post exact steps and config info?
Comment #2
jumoke commentedI am having this same problem. IE8 shows exposes unpublished comments to anonymous users, did you find a fix? CSS? Please share
Comment #3
flutedrumr commentedNo, never found a fix.
Comment #4
AlexisWilke commentedI got this error in all browsers.
The fact is that I have boost running and thus the page is cached for that Anonymous user... and then all the following anonymous users see that same cached page.
Whenever you generate that comment, you must make sure that boost does not cache the page or else... the "moderation" won't really do much good.
To me this seems to be a critical error (to not say a security issue) since an unaware admin could get spam on his site that's visible by many... (him included, except that he won't know that other anonymous users can also see the message!)
I will see if I can provide a patch for this.
Thank you.
Alexis
Comment #5
AlexisWilke commentedOkay, I have resolved the issue with Boost. Now, it is not unlikely that the previous users had a different problem since they mentioned the problem in IE8 versus other browsers.
This patch is required for anyone who runs boost.
Thank you,
Alexis Wilke
Comment #6
siliconmind commentedWell, it seems that the issue is not with the su_comments module but the boost itself. Hence it's not even a bug (at least not in case of su_comments).
Can anyone else confirm that this patch fixes this issue?
Comment #7
siliconmind commentedSince no one else is reporting this issue, let's call that one fixed.
Comment #8
AlexisWilke commentedSiliconMind,
If you think that this issue is a Boost issue, then you should bounce it off to the Boost people (by changing the Project entry of this issue).
Personally, I fixed it and it works for me. Whether you want to include the patch in your module does not matter much to me... 8-)
Thank you.
Alexis