Go to main content

Textpattern CMS support forum

You are not logged in. Register | Login | Help

#16 2013-10-01 12:44:47

Gocom
Developer Emeritus
From: Helsinki, Finland
Registered: 2006-07-14
Posts: 4,533
Website

Re: Eventual forum upgrade plan

It is of course going to stay. We will have to write a generic PHP wrapper for it and release it as a separate component, rather than keeping our own hacks. I would rather let others be able to patch it when the API changes, outside from Textpattern sphere.

The bad thing about the spam check is that you can totally circumvent it as its implemented now. Unfortunately SFS only accepts 20 000 requests a day from a single IP, after which it responds with an error page. The check relaying on active third party connection, correctly aborts on any error , but this lets any user to register.

Once a bot exhausts the quota, he can register without any worries. We can neither prevent registration since then someone could just write a nasty script that totally prevents anyone from registering.

Thing that could help would be to cache blocked IPs (ban the IP in FluxBB when the user is flagged for the first time).

Last edited by Gocom (2013-10-01 12:46:52)

Offline

#17 2013-10-01 16:43:13

ruud
Developer Emeritus
From: a galaxy far far away
Registered: 2006-06-04
Posts: 5,068
Website

Re: Eventual forum upgrade plan

Yes, definitely not keep it as a hack. Upgrading was rather painful due to all the patches that were applied.

Is that 20k limit a theoretical problem or has it occurred already?

Offline

#18 2013-10-01 17:01:25

Gocom
Developer Emeritus
From: Helsinki, Finland
Registered: 2006-07-14
Posts: 4,533
Website

Re: Eventual forum upgrade plan

ruud wrote:

Yes, definitely not keep it as a hack.

Unfortunately FluxBB employs mod culture, and doesn’t have real good plugin system (observers or stuff). Best we can do is to organize the modification well (e.g. by branching using git and merging), and patch new release. At some point this may fail tho; mod is a mod and diffs aren’t magical unicorns.

But we most certainly do have to update; there has been some pretty big security updates to FluxBB since the forum was last updated.

Is that 20k limit a theoretical problem or has it occurred already?

Hasn’t happened yet as far as I’m aware, but executing such ‘attack’ would be easy. There has been registrations from IPs/emails that have been in the spam database, but those all could as well been errors in querying SFS or alterations done after registering etc.

Last edited by Gocom (2013-10-01 17:06:27)

Offline

#19 2013-10-02 11:16:56

Gocom
Developer Emeritus
From: Helsinki, Finland
Registered: 2006-07-14
Posts: 4,533
Website

Re: Eventual forum upgrade plan

I’ve gotten the SFS protection solved on registration pages. Instead of modding FluxBB, I’ve successfully;

  1. intercepted the the registration HTTP POST requests
  2. check whether the details are in FluxBB ban table, if yes, abort and resume normal registration process
  3. otherwise check SFS database for the email and IP
  4. if found, create a new ban record to FluxBB’s database and flus cache.

FluxBB takes care of preventing access and presenting messages.

Last edited by Gocom (2013-10-02 11:17:52)

Offline

#20 2013-10-02 13:03:08

Gocom
Developer Emeritus
From: Helsinki, Finland
Registered: 2006-07-14
Posts: 4,533
Website

Re: Eventual forum upgrade plan

And I’ve gotten the log in done too. Will still need to separate the actual logic from the config file, and then that the spam protection — with zero mods.

Offline

#21 2013-10-03 07:05:54

Gocom
Developer Emeritus
From: Helsinki, Finland
Registered: 2006-07-14
Posts: 4,533
Website

Re: Eventual forum upgrade plan

Thinking about it, I’m wondering if we could do Textile support the same way as Stop Forum Spam integration; by capturing output buffer, using DOM parser to get the post body contents from the page, parse the contents and output the modified page contents. That would mean that we wouldn’t have to make any mods to FluxBB source. Wondering tho what it does to memory consumption.

Altho, the FluxBB uses its own BBCode based syntax, which will interfere with Textile.

Last edited by Gocom (2013-10-03 07:08:56)

Offline

#22 2013-10-04 11:37:14

Gocom
Developer Emeritus
From: Helsinki, Finland
Registered: 2006-07-14
Posts: 4,533
Website

Re: Eventual forum upgrade plan

Work on the forum has gone forward. Since our forum’s additional features require modifying FluxBB source, we’ve been offloading as much of it to JavaScript as possible. This includes sharing buttons, ranks and post quoting.

The new quote uses the first actual paragraph and links to the original post; this ensures quotes actually render working syntax; as we know, most quotes on these forums are unreadable and broken. Since it works by JavaScript, you are also able to quote multiple posts from one page, if needed.

New (pending) features include embedded Gist and Tweets (now using the official Twitter API), and some new ranks so that we all aren’t omegas. Must. Slip. Omega Shenron. In.

Last edited by Gocom (2013-10-04 11:41:07)

Offline

#23 2013-10-04 12:01:42

philwareham
Core designer
From: Haslemere, Surrey, UK
Registered: 2009-06-11
Posts: 3,565
Website GitHub Mastodon

Re: Eventual forum upgrade plan

Nice work Jukka! I’ll continue work on the forum CSS over the weekend.

Offline

#24 2013-10-04 12:36:38

els
Moderator
From: The Netherlands
Registered: 2004-06-06
Posts: 7,458

Re: Eventual forum upgrade plan

Wonderful, thanks guys!

Gocom wrote:

and some new ranks so that we all aren’t omegas.

Very good. A forum full of omegas looks like an old people’s home ;)

Offline

#25 2013-10-04 12:54:41

Bloke
Developer
From: Leeds, UK
Registered: 2006-01-29
Posts: 12,446
Website GitHub

Re: Eventual forum upgrade plan

Gocom wrote:

and some new ranks so that we all aren’t omegas.

Fabulous stuff. Gives me something to aim for again to try and catch up with Els :-)


The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.

Hire Txp Builders – finely-crafted code, design and Txp

Offline

#26 2013-10-04 13:08:44

uli
Moderator
From: Cologne
Registered: 2006-08-15
Posts: 4,316

Re: Eventual forum upgrade plan

Thanks for all your work, Jukka and Phil!

“Post quoting” reminded me of something: Back when the tweet links were introduced they took the position of the post numbers, which in return vanished. It’d be great if we could we have these back, there are lots of users who referenced in-topic posts with only numbers. These references are now “broken”.


In bad weather I never leave home without wet_plugout, smd_where_used and adi_form_links

Offline

#27 2013-10-04 13:41:45

els
Moderator
From: The Netherlands
Registered: 2004-06-06
Posts: 7,458

Re: Eventual forum upgrade plan

Bloke wrote:

try and catch up with Els :-)

What you need is a FluxBB plugin that splits lengthy posts into several short ones ;)

(And post more silly OT remarks like this one…)

Last edited by els (2013-10-04 13:44:05)

Offline

#28 2013-10-04 14:14:59

Gocom
Developer Emeritus
From: Helsinki, Finland
Registered: 2006-07-14
Posts: 4,533
Website

Re: Eventual forum upgrade plan

uli wrote:

Back when the tweet links were introduced they took the position of the post numbers, which in return vanished.

As we are going away from mods, these numbers are going to be back. We are probably going to leave the repeating Tweet links out and only have one for the topic, which uses the topic title as the tweet message. Posts are bit too long for a tweet.

Last edited by Gocom (2013-10-04 14:22:52)

Offline

#29 2013-10-05 12:35:44

uli
Moderator
From: Cologne
Registered: 2006-08-15
Posts: 4,316

Re: Eventual forum upgrade plan

Gocom wrote:

As we are going away from mods, these numbers are going to be back.

Great, thanks.


In bad weather I never leave home without wet_plugout, smd_where_used and adi_form_links

Offline

#30 2013-10-06 18:34:06

candyman
Member
From: Italy
Registered: 2006-08-08
Posts: 684

Re: Eventual forum upgrade plan

Offline

Board footer

Powered by FluxBB