[20200416_143747 UTC: Update 2: Added an Example to make clear what the problem was, and added 2 headings subsequent to the example to better organize the information. --martyb]
[20200416_005831 UTC; Update 1: Updated title and corrected spelling of balanceTags(). --martyb]
Ooops! Things should be working correctly, now.
tl;dr: Back on March 20th, someone tripped over a bug that appears to be in the balanceTags() routine in our Perl code. I found a way to made a quick fix to prevent its happening again, but the fix was missing a couple steps. I caught and fixed one of them, but only now just handled the other.
Workaround: When writing a comment, writing or editing a journal entry, or when submitting a story, use "DEL" instead of "STRIKE" to make text look like this.
This story is the result of something I learned in the process: properly notify the community of any changes to the site!
Symptom: It all started with a tag (i.e. HTML element) error in this comment in a user's journal where the user coded a <strike> tag, but neglected to provide a matching </strike> tag.
Example: Let's look back to the original comment which manifested this bug. Here's the latter part of it, after being corrected:
Looks who's projecting. Consider your phrase "steal jobs and send them overseas for
cheap/free labor" (no such thing as free labor). That helps billions of poor people improve their lives. Yet here you are, selfishly obsessing over your developed world pricing power (with some very unempathic label spewing) rather than display the alleged empathy or morality that you claim to be concerned about.
Your empathy is nonexistent and your morals are bankrupt - definitely not the sort of person I should be taking advice from!
The problem is that there was no closing <strike> after the word "cheap", so it looked like this:
Looks who's projecting. Consider your phrase "steal jobs and send them overseas for
cheap/free labor" (no such thing as free labor). That helps billions of poor people improve their lives. Yet here you are, selfishly obsessing over your developed world pricing power (with some very unempathic label spewing) rather than display the alleged empathy or morality that you claim to be concerned about.
Your empathy is nonexistent and your morals are bankrupt - definitely not the sort of person I should be taking advice from!
If that was all that happened it would be ugly, but tolerable. Unfortunately, every single character following it on the page was struck through, too. Not Good™.
Immediate Fix: To my knowledge there was only one way to rectify the immediate issue: manually go into the DB and insert the missing tag. This I was able to do quite quickly, but I still saw a problem.
More to Come: Anyone who saw this comment discussion, either at this moment, or who happened upon it later, would see an opportunity to intentionally leave a hanging tag and thus disfigure the site. Trolls gotta troll. So, I made the fix and noted same in this comment reply.
So, an instance of the problem was fixed, but now what? There's a "proper" way to do it, and there is another way to get the same effect that can be quickly implemented. I chose the latter.
Perl Code: Normally, such HTML errors in a user's comment or journal entry (or an editor's edit of a story!) are caught and handled by a routine in our Perl code: balanceTags(). The code looks though all the tags, with whatever nesting is present, detects where tags do not have a required closing tag,and silently inserts it into the text that makes it into the DB. It's rather hairy code because it also needs to handle: extra closing tags, mis-matched closing tags (e.g.: <b> bold <bold and italic> </b> </i>), mistyped or otherwise non-existent tags, restricting what tags are supported, and custom-created site tags! Whew!
Further, to fix it in the Perl code means going through the process of: checking the code out from GitHub, understanding the code, making the change, compiling the change, testing the change, (after rolling it out to our dev server), and then -- if all looks good -- rolling the change out to our production servers. And, of course, nobody was around at the moment who could support such activities even if it were an easy coding change (and it is not!)
Expediency: I realized there was another approach which would mitigate the problem -- not requiring Perl coding changes -- but could still prevent its recurrence: changing the value of a "Site Variable" (aka "site var").
Rehash Primer: Now I need to step back for a moment and explain a couple things. The code for SoylentNews.org is a fork of ancient Slashcode that was put up on GitHub. Slashcode was implemented using a Model View Controller design. There is a clear demarcation between what is done where and at what level.
Templates: As part of its implementation, the SlashCode implemented "Templates" which generate the HTML pages that get sent to the browser and act as an interface between the code and the user. As far as I know, every page you see on the site comes by way of a template. Each template may, in turn, make use of other templates. Templates can make calls to underlying Perl code. This is where the site implements the heavy lifting of talking to the database (DB), creating e-mails, and other closer-to-the-metal activities. The template language (from personal inspection; I have yet to find an official document as to its syntax and semantics) presents what appears to be a simple, macro-capable language. The templates are stored in the DB and loaded into memory when the site is started. An advantage of this is that changes to templates can be made "on the fly" using a template editor (which is, itself, a template!) There is one caveat: for the changes to take effect, processes on the front-end servers need to be "bounced", i.e. restarted, so the changes are loaded into memory from the (updated) DB.
Site Variables: There are some parameters whose values affect the site's operations: Name of the site, domain name of the site, the name of the Anonymous User account, ... it goes on and on and on. There are no less than 750 site variables! And, as many things that grew beyond their initial construction, there is no simple way to look for what site vars might be appropriate to any given situation. One is just expected to know what they are and what they do and how they do it. Simple enough approach when they first started, I guess. A search capability would be very nice to have, but it will take some coding to make that happen, so it has become just another of the several changes that would be nice to make to the site.
So, back to the matter at hand, I knew about the "approvedtags" site var which lists all tags which are permitted to be used on the site. Sure enough, "STRIKE" was in there! And, I saw that "DEL" was in there, too. Does "DEL" have the same problem? I tried a quick test comment on our development server and it revealed that balanceTags() properly handled a hanging <DEL> without a matching </DEL>. Yay! I removed "STRIKE" from the "approvedtags" list, saved the change, bounced the front-end servers, and breathed a sigh of relief.
All was good, until someone asked in a footnote to a comment why do we still list STRIKE as being a permitted tag for comments? What? I double-checked and verified that "STRIKE" was no longer listed in "approvedtags". What is going on? So, I commenced searching and finally discovered another site var: "approvedtags_visible" which contains the list of tags that is presented to the user as being available. And, sure enough, "STRIKE" was in that list. Grrr! I removed "STRIKE" from "approvedtags_visible", saved the changes, and saw no further issues mentioned there. Finally!
Or so I thought. Did you see what was missed? The site vars were now correct and up-to-date. The changes were saved to the DB. But... those changes existed only in the DB. Still needed to 'bounce' the front end servers for the changes to take effect. So, that entailed a quick SSH to our servers, running the bounce scripts, and verifying that "STRIKE" was truly and properly removed from the tags presented to the user as being available for use, and that anyone trying to use <STRIKE>, anyway, would discover it did not work.
Conclusions: So, here are some lessons learned:
- When you want to use a <STRIKE> tag, use <DEL>, instead.
- There is no assurance that reporting a problem in the comments will be noticed.
- Please report site issues with an e-mail sent to admin (at) soylentnews (dot) org.
- In addition to sending an e-mail, mention it in the "#dev" channel of our IRC server.
- This particular issue should now be well and truly fixed. Please report any problems you may discover with it.
- When communicating changes made to the site, a mention in the comments does not suffice.
--martyb
[Janrinok says: TL:DR Martyb fixed it, OK, OK, I have read it....]
Related Stories
At least one person in Kentucky is infected after taking part at a "coronavirus party" with a group of young adults [...]
The partygoers intentionally got together "thinking they were invincible" and purposely defying state guidance to practice social distancing, [...]
[...] the virus seems to be affecting young people in the United States more than it has in China. A report released last week by the Centers for Disease Control and Prevention showed that up to 20% of people hospitalized with coronavirus in the United States are between the ages of 20 and 44.
[...] "So far the demography definitely seems to be very different in the United States versus in other countries that saw this hit earlier,"
[...] In New York state, more than half of coronavirus cases -- 53% -- have been among young people between the ages of 18 and 49
From MSN:
Kentucky coronavirus party with group of young adults has left at least one person infected:
At least one person in Kentucky is infected after taking part at a "coronavirus party" with a group of young adults [...] The partygoers intentionally got together "thinking they were invincible" and purposely defying state guidance to practice social distancing [...] "This is one that makes me mad," the governor said. "We have to be much better than that."
And...From Slate:
A group of Kentucky partygoers recently attended a "coronavirus party." The event, which appears to be a pandemic-themed soiree, as you might imagine, was not a civic-minded effort to promote social distancing practices and best hand-washing practices, but a slap in the face to everyone else's collective efforts to not kill our parents and grandparents. The party mocked the virus, and the coronavirus gods were angry. One of the twentysomething attendees of the ill-advised gathering in the midst of a national emergency tested positive for the virus Tuesday.
Maybe I'm too old to get it, but it seems to me somewhat unwise to do this.
(Score: -1, Offtopic) by Anonymous Coward on Wednesday April 15 2020, @10:31PM (2 children)
That's nice and all, but any of you schumucks actually got trump bucks? It's been a while since I've got my obama bucks, and my stash needs replenishing.
(Score: -1, Troll) by Ethanol-fueled on Thursday April 16 2020, @12:00AM
Hell yeah nigger, I got 1200 Trumpbucks and I finna spendit on Henney and Swisher sweets. And a nice new suit to be hired, but nobody hirin' a White boy. They sneaking stinky street-shittin' Indians and Mexicans thro' the back do' like dey alway do.
(Score: 0) by Anonymous Coward on Thursday April 16 2020, @12:18AM
Yeah, the Trumpbux are real.
Maybe we can just make this a monthly thing, like some kind of universal disaster relief income.
(Score: 0) by Anonymous Coward on Wednesday April 15 2020, @10:41PM (2 children)
Oh cool i tried using strike the other day, was disappointed, did not report it of course.
(Score: 0) by Anonymous Coward on Wednesday April 15 2020, @11:41PM
Used it earlier... checked allowed tags and replaced with del. I did wonder for a second if I was going Biden but it seems I'm safe and SN was gaslighting me - no refunds!
(Score: 0) by Anonymous Coward on Thursday April 16 2020, @02:47PM
test...
shit strike no worky
shitdel worky worky...end test
(Score: 1) by AlwaysNever on Wednesday April 15 2020, @10:49PM
So are you still on Gentoo? If so, how is life with it in the long run? Are you ready to embrace SystemD?
(Score: 1, Insightful) by Anonymous Coward on Wednesday April 15 2020, @11:44PM (11 children)
<sarcasm>You're all such good coders, choosing to remove the strike tag instead of fixing the balance_tags() function. I'm in awe of the coding skills of the admins here.</sarcasm>
(Score: 0, Troll) by Ethanol-fueled on Thursday April 16 2020, @12:04AM
It looks like it's time to look at the code and become a true site-knowledgeable nigger. Specifically at the database and what a "row" looks like.
(Score: 5, Touché) by martyb on Thursday April 16 2020, @01:17AM (8 children)
As was made clear in the story, something was needed in a hurry. Also, no claim was made in the story about our being "such good coders". I certainly make no claims as to my programming skills being exemplary. As we are all pretty strapped for time here, you're more than welcome to look at the code [github.com]. Then just search for "balanceTags".
<sarcasm>We look forward to your submitting a pull request and showing us how it should be done! =)<sarcasm>
Wit is intellect, dancing.
(Score: 5, Funny) by deimtee on Thursday April 16 2020, @01:27AM (3 children)
You forgot to close the <sarcasm> tag. You could send the whole site sarcastic.
Here, I'll fix it. </sarcasm> </sarcasm> </sarcasm>
Now everything will be sweetness and sincerity.
If you cough while drinking cheap red wine it really cleans out your sinuses.
(Score: 3, Funny) by sjames on Thursday April 16 2020, @04:11AM
Yes, I'm certain that will fix everything....NOPE, didn't work!
(Score: 3, Touché) by martyb on Thursday April 16 2020, @03:11PM
Wit is intellect, dancing.
(Score: 3, Informative) by DannyB on Thursday April 16 2020, @06:53PM
<no-sarcasm>
In case you haven't noticed, I use no-sarcasm tags when appropriate.
</no-sarcasm>
The people who rely on government handouts and refuse to work should be kicked out of congress.
(Score: 3, Insightful) by Reziac on Thursday April 16 2020, @02:23AM (1 child)
Or a good case of... a timely fix is better than a perfect fix.
Thanks for keeping SN up and running. From where I sit, it works great.
And there is no Alkibiades to come back and save us from ourselves.
(Score: 3, Insightful) by martyb on Thursday April 16 2020, @07:26PM
Wit is intellect, dancing.
(Score: 3, Funny) by barbara hudson on Thursday April 16 2020, @02:59AM (1 child)
For bonus points, submit it in COBOL. I hear there's a huge demand.
No rush, it's practice for an urgent government project in Florida, so any time this decade is okay.
SoylentNews is social media. Says so right in the slogan. Soylentnews is people, not tech.
(Score: 0) by Anonymous Coward on Thursday April 16 2020, @10:33PM
What's FUNNIER than tranzoid barb's bs above? HOW BADLY I BLEW AWAY THE BULLSHIT ARTIST "BOoooOOooRRRrrPPpp" here https://soylentnews.org/comments.pl?noupdate=1&sid=33430&page=1&cid=889582#commentwrap [soylentnews.org] & then SCUMBAG Abomination of DESOLATION twisto barbara hudson TRIED to FAKE that I "apologized" that THAT DISGUSTING MONSTROSITY "it" is here https://soylentnews.org/comments.pl?noupdate=1&sid=35327&page=1&cid=937346#commentwrap [soylentnews.org] ?
* RoTfLmAo...
(I notice you've GIVEN UP trying to STOP ME POSTING too losers - is it since you can't? That's right - unstoppable here)
APK
P.S.=> You PACK of UNBELIEVABLY WEAK little fuckboys, lol - you make me laugh... apk
(Score: 2) by The Mighty Buzzard on Thursday April 16 2020, @12:39PM
You lot may be at home sitting around eating bonbons, watching Oprah, and spending your trumpbux on amazon but I don't get that luxury. No non-emergency coding for SN is going to happen until I'm done with and relocated into the church unless someone else wants to do it. I have entirely too much to do and time is very much a constraint. Which is to say, submit a pull request or bite me.
My rights don't end where your fear begins.
(Score: -1, Troll) by Anonymous Coward on Wednesday April 15 2020, @11:45PM
Spread the word of ID2020. Warn your family and friends.
Do not take "the mark" no matter where it comes from or how it is delivered.
Reject a tattoo or chip or whatever they want to MARK you with so you can crawl out of your house, take the cheetoes out of your hair and pretend to be something you're not again.
Spend your "Stimulus" check(s) on the homeless.
(Score: -1, Spam) by Anonymous Coward on Wednesday April 15 2020, @11:56PM (3 children)
See subject & I apologize for my spam the past few days. I have SERIOUS mental illnesses & sometimes can't control my RAGE so it comes out online.
I have no family & they all left me for good. I have no job & I'm WORRIED how I'm going to pay my bills w/ the economy tanking. I reverted to my old ways instead of facing my problems & slipped into my old habits of shitposting. I got angrier when I got spamdownmodded & threatened to stop the site owner's server in a fit of RAGE.
* I'm sorry martyb & barbara hudson. I'm the problem & sometimes just can't control my mental illness and HATEFUL impulses. Being stuck ALONE in my house hurts even more. I used to go walking & get exercise to stay under 350 lbs but now I'm stuck inside with this damned STAY AT HOME ORDER in New York. I need help w/ my mental health but can't go see my therapist because of coronavirus. I'm just NOT in a good state of mind & took it out on all of you.
Please forgive me & let me post about APK Hosts File Engine w/o getting spamdownmodded. It's my life's work & helps me to feel useful so I don't get depressed and full of rage. I feel worthwhile when I post about HOSTS & don't feel so useless.
APK
P.S.=> I think you all know this but I couldn't REALLY take SN offline. That's just idle bluster when I'm mad... apk
(Score: 2) by hendrikboom on Thursday April 16 2020, @02:06AM (1 child)
Try a phone session with your therapist. I have family that finds therapy works better by phone. Might work for you. Might be better than having no therapy.
-- hendrik
(Score: 0) by Anonymous Coward on Thursday April 16 2020, @10:50PM
APK did a real number on barbara hudson here technically and catching her impersonating him by quote of it https://soylentnews.org/comments.pl?noupdate=1&sid=33430&page=1&cid=889582#commentwrap [soylentnews.org] and as far as marty blowbrain? Too much coke will pop your cork and give you a stroke aneurism now won't it?
(Score: 0) by Anonymous Coward on Thursday April 16 2020, @10:38PM
Dear TRANZOID MONSTROSITY Abomination of DESOLATION: This about SUMS UP your SHAME fucker https://soylentnews.org/meta/comments.pl?noupdate=1&sid=37083&page=1&cid=983818#commentwrap [soylentnews.org] (notice you stopped posting after you attempt to IMPERSONATE me not only here but also from DEC 30 2019 I put up in that link ALONG WITH WHERE I SHATTER YOU ON EVERY LEVEL THERE IS in tech & on your bullshit INCLUDING YOU TRAPPED BY QUOTE from slashdot where you told everyone to STALK/HARASS me too https://soylentnews.org/comments.pl?noupdate=1&sid=33430&page=1&cid=889582#commentwrap [soylentnews.org] )
* LMAO!
APK
P.S.=> DISGUSTING Abomination of DESOLATION estrogen OD'd MALE faking FEMALE you - you are SUCH A TOTAL FAIL & that link above + the ones it points to PROVE it all for me, loser... apk
(Score: 3, Interesting) by Muad'Dave on Thursday April 16 2020, @11:53AM (8 children)
Out of curiosity, why did you feel the need to remove the strike tag in the first place? It's a lot more intuitively-named than 'del'.
(Score: 2) by martyb on Thursday April 16 2020, @02:45PM (7 children)
Well it is right there in the write-up... well, not as clear as it could have been. I've added an example which should help clarify what happened and why a fix was needed.
Thanks for asking!
Wit is intellect, dancing.
(Score: 2) by Muad'Dave on Thursday April 16 2020, @03:35PM (6 children)
I guess I still don't understand why you removed the tag instead of just fixing it. How was strike any different than del? I see that strike is deprecated [mozilla.org], but you have <del> but not <s> [mozilla.org]?
Del
does this, but s does nothing.(Score: 2) by maxwell demon on Thursday April 16 2020, @04:40PM (1 child)
Because the code as is supports del, but not strike. And the tag names are hard-coded in that function.
The Tao of math: The numbers you can count are not the real numbers.
(Score: 2) by Muad'Dave on Friday April 17 2020, @11:40AM
Thank you - that's the answer I was looking for.
(Score: 3, Informative) by martyb on Thursday April 16 2020, @07:09PM (3 children)
Over the years, new HTML Tags have been proposed and implemented. Any code that is designed to deal with HTML Tags needs to be able to handle whatever new Tags may come along. "It's difficult to predict things, especially the future!" So, rather than accept everything that comes along and then start blacklisting [wikipedia.org] new, unsupported Tags as they appear, the code instead uses a form of whitelisting [wikipedia.org] to specifically enumerate the Tags it can properly deal with.
It is important to note that there are two different kinds of "code" here.
On the one hand, there is the Perl code which actually implements the guts of balanceTags(). All Tags are not created equal! Some Tags work at a BLOCK level and some are IN-LINE. Some require closing Tags, some do not (eg <br>, <br />, and <hr>). Then there are Lists which can contain certain elements, but those elements are not permitted standalone. For example, a <LI> is only permitted within a <UL> or <OL> block. And then again, <DD&rt; and <DT&rt; are only permitted within a <DL&RT;. Then there are limits that one might want to impose such as how many times one wants to permit, say, superscripts or subscripts. e.g. 2<sup>3<sup>4<sup>5</sup></sup></sup> is rendered here as 2345; the final level of superscripting is supressed. (Actually, that is yet another Site Var which limits the maximum supported nesting!)
In short, the Perl code needs to "know" these things, and what to do with them, and have the code that actually makes it all happen.
On the other hand, the site presents Site Variables (Site Vars) which serve to turn the knobs, if you will, of what the Perl code is permitted to do. Imagine a speed limiter on a sports car or on a truck/lorrie. So, the Perl code implements what it does, and has its whitelists of how to support various Tags. The Site Operator, (previously: Slashdot; currently: SoylentNews) may have a desire to present only a subset of those. This is where the Site Vars: approvedtags and approvedtags_visible come into play. The Site Operator can "spin the knobs" and restrict which Tags are made available to their users. Just by changing the values of these Site Vars, they could, for example, reduce the apparent complexity of the site to the users and limit the permitted tags to just <P>, <P>, <B>, <I>, and <A>. Paragraphs, Bold text, Italics, and Anchors (aka links). We are a tech-oriented site so pretty much all the tags that the Perl knows how to deal with are whitelisted.
The problem arose, apparently, when someone added <STRIKE> to the approvedtags and approvedtags_visible Site Vars... but failed to ensure that <STRIKE> tag was "known" to the Perl code.
So, here's the deal. The Perl had data structures that explicitly supported <DEL> as a block Tag. So, it "knew" that if it ever encountered a <DEL> with a matching </DEL>, it should internally provide one in the proper place. It DID NOT, however, explicitly list <STRIKE> as one it knew what to do with. And, so long as it was allowed through by the Site Vars, AND the user always provided the closing </STRIKE> Tag, everything worked like a champ! But, as soon as someone failed to explicitly provide their own </STRIKE> in their comment... Bad Things Happened.
Be aware the preceding is all based on what I have been able to glean from looking over the site; I could well have something confused. But, this is my working knowledge of how things work and I can assure you the code has no shortage of oversights and gotchas. We have come a LONG ways from back at the start of SoylentNews when the site crashed, hard, several times every day! I continue to be in awe of what was successfully wrought from an out-of-date and unsupported code base to make this site what it is today.
Wit is intellect, dancing.
(Score: 2) by Muad'Dave on Friday April 17 2020, @11:38AM (2 children)
I understand the technical challenges of implementing the tag logic (in perl???).
I'm asking a much simpler, non-technical question - why was the tag named strike removed in favor of the less descriptive 'del' tag? What drove the decision to deprecate the 'strike' tag instead of using that name over 'del' for the desired functionality?
(Score: 2) by martyb on Friday April 17 2020, @01:52PM
DEL is already is in the Perl code.
DEL works and remains available.
If STRIKE is not in the Perl code, it will not work correctly.
STRIKE has been made unavailable.
Wit is intellect, dancing.
(Score: 2) by martyb on Saturday April 18 2020, @04:07PM
I may have misunderstood the question, so let me try something else.
Question: So, when they wrote the Perl code for balanceTags(), why did they explicitly support DEL and fail to explicitly support STRIKE?
Answer: Good question! I am not sure why. Most likely? Just simple human error.
What's next? When time permits, we will go back, review the situation, and likely add support for STRIKE into the Perl code, enable it in the Site Vars, and all will be rainbows and unicorns. =)
What follows is my best guess as to why STRIKE was not explicitly included.
Some History: What you are looking at now, in your browser, is the product of what is stored in our source code repository as "rehash". It was built upon a fork of the most-recent-at-the-time version of slashcode which was purported to run Slashdot. I suspect it was intended to be, at least. But, it was not maintained and by the time SoylentNews was getting started, slashcode was very much out of date. It had dependencies on versions of things like apache and Perl and MySql that were no longer supported. Those who started SoylentNews worked like crazy to bring things up to the current day, fixed bugs in what they inherited, and finally started adding our own improvements to it. So that is what we have as out code base, now. Rehash.
Realize that Slashdot got started in October of 1997. Back then, HTML was at version 3.2 or maybe 4.0 or thereabouts. A bunch of new HTML tags have been added since then. Though it looks like they tried to follow good development practices, it sure looks to me that some coding fixes were made less carefully. I started using Slashdot before they even instituted usernames and UIDs. So I witnessed some "situations" where "Trolls" tried to break the site. For example, I remember when page-widening trolls were a thing. If a single word in a comment was long enough (say, over 80 characters), it would result in the "comment box" being wider than the page it was on. The browser would do its best to display things, but it really made a mess of that entire page. It forced the user to have to scroll left and right to read each comment. So, there was not a lot of time to plan, design, and implement a fix. They figured something out and pushed it out to the site in a matter of a day or two. Things seemed to work after they made a couple more fixes. But I seriously doubt that they went back and tested all the potential ramifications of that change.
When you consider they had a site with millions of users back then, there was a lot of pressure to fix things ASAP!
That's just an example. Many more on-the-fly fixes were added to the code, deemed to seem to work, and called "good enough" until proven otherwise.
So, way back when balanceTags() was written, they were trying to fix a problem that had arisen when someone wrote a comment and failed to close a block tag that they had started. IIRC, that was with one of the more common tags like <B> for BOLD text. All text on the page from that point onward would be in bold. Same problem with <I> for ITALIC text.
Whatever the reason, they failed to include STRIKE. To add it, now, would require all the stuff I mentioned in an earlier comment. Suppressing STRIKE and allowing continued use of DEL which was explicitly included in the Perl code was possible with no Perl coding changes. Given the situation at hand, that path was taken for now. When time permits, we will go back, specifically add STRIKE to the Perl code, test it, and then if all is good, finally push it out to the production server.
Hope that helps!
Wit is intellect, dancing.
(Score: 3, Informative) by maxwell demon on Thursday April 16 2020, @12:52PM (5 children)
Thanks for the explanation.
BTW, I now got curious and looked into the balanceTags function on GitHub, and my guess is that to make <strike> work, you would have to add it to the variables %known_tags (line 3016) and %no_block (line 3032).
Not that it really matters, given that <del> gives the functionality anyway (and even is less to type).
The Tao of math: The numbers you can count are not the real numbers.
(Score: 2) by martyb on Thursday April 16 2020, @02:54PM (4 children)
Thanks for taking a look and finding that!
Now, picture yourself on the site, with strike characters running all the way to the end of the page. And knowing there's gotta be that one troll who would see this and run with it, posting a comment with an unclosed strike to every story. And the only way to fix each instance is a manual DB update. Whack-a-troll is not my idea of a good time. When I realized there was a quick fix, I took it and pushed it out ASAP. And, of course this was while a few other things were swirling around at the same time. And as you saw in the write-up, and TMB's comment to this thread, making a coding change under time pressure and rolling it out to the site seemed like a very not good idea.
Thanks again for looking at the code and coming up with that fix!
Wit is intellect, dancing.
(Score: 2) by maxwell demon on Thursday April 16 2020, @04:16PM (3 children)
My comment was in no way meant to imply you were lazy or something like that. After all, I didn't even test that this would indeed fix the problem (that would have required me to set up a test server just for this), I just looked at the code and figured this should do it. No guarantee that I didn't overlook something.
As I wrote, I just got curious, so I looked. And then I thought there's no reason not to share my result.
The Tao of math: The numbers you can count are not the real numbers.
(Score: 2) by martyb on Thursday April 16 2020, @06:01PM (2 children)
My bad! In no way did I mean to suggest any shortcoming on your part! In fact, I am grateful for your looking at the code and your suggested fix!
My intention was to provide some background on why *I* chose to make the fix the way I did!
And, yes, I cam to the same conclusion as you did WRT the code, under the assumption that the comments were correct and all the other code was functioning as it should. I am by no means conversant with all the two-symbol data structure abstractions that Perl seems to relish!
And, yes, implementing the change, compiling the code, rebuilding the system, rolling it out, and then testing it... that was going to have to wait for some time when more time and less pressure for a fix is available.
I humbly apologize for any offense taken; absolutely NONE was intended!
Thanks again for looking at the code and your suggested fix!
Wit is intellect, dancing.
(Score: 2) by maxwell demon on Thursday April 16 2020, @07:09PM (1 child)
Don't worry, you haven't offended me. Rather I thought you misunderstood me in a way that would offend you, so I wanted to make clear that it was not meant that way. Sorry for all the misunderstandings I apparently caused.
The Tao of math: The numbers you can count are not the real numbers.
(Score: 2) by martyb on Thursday April 16 2020, @07:24PM
Wit is intellect, dancing.