Decontaminator by Flarumite

Rail

Very nice addon.
Would be great to be able to flag posts for approval and for moderator review. (Seperately)
😃

And, i think this is very hard to implement,
but certain groups should be able to see what was written before the system changed it..
Maybe a feature for the future 🙂

IanM

Rail Would be great to be able to flag posts for approval and for moderator review. (Seperately)

Great minds think alike 😉 I do have in mind adding support for flarum/approval in the future

Rail And, i think this is very hard to implement,
but certain groups should be able to see what was written before the system changed it..
Maybe a feature for the future 🙂

Interesting idea, let me give it some thought!

[deleted] This is a brilliant extension, and could well be used as a way of redacting data that falls under GDPR automatically

Thank you 🙂 Precisely, I'm actually using it to protect against posting of credit card numbers, email addresses and general PII too

[deleted] I hope nobody in the UK actually has 07123456789 as a mobile number

I certainly hope not! In theory no, as 071 I don't think is assigned to cellular numbering groups

[deleted] is "flarumite" a play on "marmite"

It's been stuck in my head for a while now, can't remember where it came from (perhaps @Ralkage as he mentions). Just thought it was a reasonable publisher name, and it was available!

[deleted]

@IanM This is a brilliant extension, and could well be used as a way of redacting data that falls under GDPR automatically. A couple of points from me

I hope nobody in the UK actually has 07123456789 as a mobile number 🙂
is "flarumite" a play on "marmite" - you either love it, or hate it ? 🙂

Thanks again for this - it's truly a game changer.

Ralkage

[deleted] to answer point 2, it's been an ongoing thing in this community and origin/creation date still unknown 🤣 I believe one of us on the staff started using it in our releases.

Edit: Looks like I started that trend, whoops 😆

[deleted]

Hmmm

**phenomlab@vps:~/hub.phenomlab.com$ composer require flarumite/flarum-decontaminator
Using version ^0.1.1 for flarumite/flarum-decontaminator
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - The requested package fof/pages (locked at 0.4.0, required as ^0.3.0) is satisfiable by fof/pages[0.4.0] but these conflict with your requirements or minimum-stability.
  Problem 2
    - The requested package fof/sitemap (locked at 0.4.0, required as ^0.3.0) is satisfiable by fof/sitemap[0.4.0] but these conflict with your requirements or minimum-stability.


Installation failed, reverting ./composer.json to its original content.
**

clarkwinkelmann

[deleted] the error seems unrelated to this extension. It seems you somehow managed to install version 0.4.0 of both fof/pages and fof/sitemap without updating the requirement in composer.json. Running composer require fof/pages fof/sitemap might help.

[deleted]

clarkwinkelmann ok, but...

phenomlab@vps:~/hub.phenomlab.com$ composer require fof/pages fof/sitemap
Using version ^0.4.0 for fof/pages
Using version ^0.4.0 for fof/sitemap
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Nothing to install or update
Package league/url is abandoned, you should avoid using it. Use league/uri instead.
Package mtdowling/cron-expression is abandoned, you should avoid using it. Use dragonmantank/cron-expression instead.
Writing lock file
Generating autoload files
Carbon 1 is deprecated, see how to migrate to Carbon 2.
https://carbon.nesbot.com/docs/#api-carbon-2
    You can run './vendor/bin/upgrade-carbon' to get help in updating carbon and other frameworks and libraries that depend on it.
62 packages you are using are looking for funding.
Use the `composer fund` command to find out more!

clarkwinkelmann

[deleted] that's looking good! Are you able to install other extensions now ?

The important part in the output is composer.json has been updated, which should solve the issue you previously had.

[deleted]

clarkwinkelmann It does indeed - thanks !!

phenomlab@vps:~/hub.phenomlab.com$ composer require flarumite/flarum-decontaminator
Using version ^0.1.1 for flarumite/flarum-decontaminator
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Package operations: 1 install, 0 updates, 0 removals
  - Installing flarumite/flarum-decontaminator (0.1.1): Downloading (100%)         
Package league/url is abandoned, you should avoid using it. Use league/uri instead.
Package mtdowling/cron-expression is abandoned, you should avoid using it. Use dragonmantank/cron-expression instead.
Writing lock file
Generating autoload files
Carbon 1 is deprecated, see how to migrate to Carbon 2.
https://carbon.nesbot.com/docs/#api-carbon-2
    You can run './vendor/bin/upgrade-carbon' to get help in updating carbon and other frameworks and libraries that depend on it.
62 packages you are using are looking for funding.
Use the `composer fund` command to find out more!

[deleted]

IanM Thank you 🙂 Precisely, I'm actually using it to protect against posting of credit card numbers, email addresses and general PII too

As a CISO / DPO / Privacy Advocate in my professional capacity, GDPR obviously extends into my own personal phenomlab.com forum, so this is very welcome from me. It's something that is essential.

IanM I certainly hope not! In theory no, as 071 I don't think is assigned to cellular numbering groups

Just kidding 🙂

IanM Just thought it was a reasonable publisher name, and it was available!

Hard to argue that one - it 100% fits the bill

[deleted]

Despite a validated regex of /^([a-zA-Z0-9_\-\.]+)@([a-zA-Z0-9_\-\.]+)\.([a-zA-Z]{2,5})$/m I'm struggling to get this to work. The post is created as usual, but the email address I supply isn't being replaced - just ignored. Not sure if this is related to ReCache or not ?

IanM

[deleted] are you posting from a user that does not have the bypass permission by any chance?

IanM

[deleted] Despite a validated regex of /^([a-zA-Z0-9_\-\.]+)@([a-zA-Z0-9_\-\.]+)\.([a-zA-Z]{2,5})$/m I'm struggling to get this to work.

So it looks like it's the ^ and $ (start of line, end of line) that is causing the problem here. Was that intentional?

If you create a post with only an email address in the content or title, does that match using your regex? It does for me on test...

[deleted]

IanM Oh, sh... yes - admin....my fault....

I'll need to create a test user for this 🙂

[deleted]

IanM Well, I just created a vanilla user, and it's the same issue. This user is simply a member, but is still able to post his email address.

IanM

[deleted] give me a few minutes (I’m on mobile at the moment). I’ll try your regex locally...

In the meantime, does the sample rule included with the extension work for you?

[deleted]

IanM Can't say for sure, as I deleted it 🙂 Sorry !

IanM

[deleted] I've confirmed that your regex, although it does validate does not match via the extension. I'm still looking into it tho.

This is the regex I use for email matching, which does work

/\b[a-z0-9._%+-]+@[a-z0-9.-]+\.[a-z]{2,4}\b/mi

Would you mind trying that one for the moment?

[deleted]

IanM yes, it was intentional as that way it would only match the email address, but I was using a paragraph previously without the additional symbols. Let me play with this a bit more

Thanks

[deleted]

IanM Just circling back here to let you know that all is working as expected. Performed various tests this morning and all work.

I have a suggestion though. Is it possible to add a class (style) to the replacement text ? For example, something like "[redacted for privacy reasons]" should stand out a bit more - for example

[redacted for privacy reasons]
[redacted for privacy reasons]
[redacted for privacy reasons]

With a CSS style, it'd be possible to much more with the text I think which would be an effective enhacement.

IanM

[deleted] Just circling back here to let you know that all is working as expected. Performed various tests this morning and all work.

Excellent, I'm glad it's working out for you now 🙂

[deleted] I have a suggestion though. Is it possible to add a class (style) to the replacement text ?

That's a fantastic idea. I've created an issue, and I'll start looking into it flarumite/flarum-decontaminator3

IanM

[deleted] I have a suggestion though. Is it possible to add a class (style) to the replacement text ?

I've been playing around with this idea a little, and would value your opinion on this, please:

Replaced content is wrapped in a span, with a class assigned to it. Also added a tooltip with translatable string.

Is this along the lines of what you had in mind?

« Previous Page Next Page »