Page MenuHomeFreeBSD

add plausible analytics to main site and docs
ClosedPublic

Authored by mark_freebsdfoundation.org on Apr 10 2025, 12:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 15, 10:01 AM
Unknown Object (File)
Sun, Jun 15, 3:44 AM
Unknown Object (File)
Mon, Jun 9, 2:50 PM
Unknown Object (File)
Mon, Jun 9, 9:05 AM
Unknown Object (File)
Fri, Jun 6, 4:41 AM
Unknown Object (File)
Fri, Jun 6, 1:38 AM
Unknown Object (File)
Wed, Jun 4, 1:19 AM
Unknown Object (File)
Tue, Jun 3, 12:27 AM

Diff Detail

Repository
R9 FreeBSD doc repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Hey folks. This is the stats I'd like to add to the website in the simplest and quickest way possible. With a couple of upcoming blog series we have planned, it would be fantastic to see how they land across the main, doc and foundation websites. Thanks!

per discussion in core@ this is OK, but should get an OK from either clusteradm or secteam, due to the 3rd party dependency. Other than that LGTM.

There are options for self-hosting the javascripts and proxying the events back https://plausible.io/docs/proxy/guides/nginx if necessary.

This revision is now accepted and ready to land.Apr 10 2025, 12:38 PM

added clusteradm or secteam per review of 3rd party dependency.

We could add SRI hash (as I do with my own site, running Hugo) — which requires monitoring for changes. I have that in place on my site (cronjob, perl — I'm old — checker) and would be happy to monitor it here too. Extra work, but I've no problem 'owning' that.

bcr added a subscriber: bcr.

Adding doceng@ for completeness sake, since the website is their domain. If core already approved, that's fine then. Just want to make sure everyone is involved.
I like the change as it is both benefitting us in learning a bit more about our visitors, while having privacy as a main focus.

Just to clarify, the approval is just for the technical part (build and run) of the website.
I do not enter to discuss the privacy implications.

waiting on doceng internal discussion.

This revision now requires review to proceed.Apr 25 2025, 10:36 AM

@fernape: Just to check, since you're a doceng member: was your approval made in your own name or as a doceng member (for the team)?
The latter would remove the blocker for this and we can go ahead. If the team needs more time, then let us know.
Thanks!

gordon added a subscriber: gordon.

I’m not entirely sure what kind of approval from secteam is being sought. If someone in core would like to help me understand what kind of review is expected, I’d be happy to undertake it.

Until then, I don’t want to be the blocker for this otherwise reasonable sounding change.

In D49757#1147818, @bcr wrote:

@fernape: Just to check, since you're a doceng member: was your approval made in your own name or as a doceng member (for the team)?
The latter would remove the blocker for this and we can go ahead. If the team needs more time, then let us know.
Thanks!

My approval was just from the doc technical point of view.
doceng@ had a discussion about this change and sent core@ an answer about this via the doceng@ liaison.

In D49757#1147818, @bcr wrote:

@fernape: Just to check, since you're a doceng member: was your approval made in your own name or as a doceng member (for the team)?
The latter would remove the blocker for this and we can go ahead. If the team needs more time, then let us know.
Thanks!

My approval was just from the doc technical point of view.
doceng@ had a discussion about this change and sent core@ an answer about this via the doceng@ liaison.

OK, thanks for the answer, @fernape!
It seems like this is in core's hands. @dch If you guys agree this is good to have, then we should move forward with it.

I’m not entirely sure what kind of approval from secteam is being sought. If someone in core would like to help me understand what kind of review is expected, I’d be happy to undertake it.

Until then, I don’t want to be the blocker for this otherwise reasonable sounding change.

Thanks Gordon, its whether there were any concerns around including 3rd party javascripts on our website. I believe you're OK with this based on comment.

Per core 2025050 meeting we agreed to proceed with this change.
Thanks all for your well-considered & cordial discussion.
I will amend the doceng to non-blocking, and commit the diff.

dch, on behalf of core@

This revision is now accepted and ready to land.May 13 2025, 8:54 AM
This revision was automatically updated to reflect the committed changes.
OSZAR »