Skip to content

feat: add docs pages for DLP - #305

Merged
bsmth merged 7 commits into
mozilla:mainfrom
bsmth:cta-dlp
Sep 1, 2026
Merged

feat: add docs pages for DLP#305
bsmth merged 7 commits into
mozilla:mainfrom
bsmth:cta-dlp

Conversation

@bsmth

@bsmth bsmth commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Description:

Adds two reference pages for DLP

Motivation:

Making sure we have docs pages for new policies.

Related issues and pull requests:

Follow-up from:

@bsmth
bsmth requested a review from jonathanmendez September 1, 2026 07:33
Comment thread src/plugins/inject-oma-uri.mjs Outdated

@jonathanmendez jonathanmendez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a few minor suggestions.

Comment thread src/content/docs/reference/policies/ContentAnalysisTelemetry.mdx Outdated
Comment thread src/content/docs/reference/policies/DataLossPrevention.mdx Outdated
Comment thread src/content/docs/reference/policies/ContentAnalysisTelemetry.mdx Outdated
Enable and configure logging of Data Loss Prevention events.

When a DLP rule is triggered, Firefox records the event and submits it in the `enterprise` telemetry ping, which is kept separate from general Firefox telemetry.
Events are logged with full URLs by default, so this policy is normally used to turn logging off or to reduce what it records.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the default behavior (at least being enabled, but potentially the default URL level of detail) may change with Bug 2067262. I don't know if it's worth settling when/whether the bug will be implemented before stating the default behavior here?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's update this page when the "default off" implementation lands so that we keep docs reflecting current state (reality) and there's less chance of drift from what's actually shipping, do you agree?

Comment on lines +27 to +28
- `Enabled`: A Boolean. If `false`, DLP events are not recorded. The default is `true`.
- `UrlLogging`: Controls how much URL information each event records. The default is `full`.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto about default behaviors potentially changing

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

Comment thread src/content/docs/reference/policies/ContentAnalysisTelemetry.mdx Outdated
Comment thread src/content/docs/reference/policies/DataLossPrevention.mdx Outdated
Comment thread src/content/docs/reference/policies/DataLossPrevention.mdx Outdated
Co-authored-by: Jonathan Mendez <jonathanmendez@users.noreply.github.com>
Co-authored-by: Brian Smith <brian@smith.berlin>
@bsmth
bsmth requested a review from jonathanmendez September 1, 2026 19:38
@bsmth

bsmth commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator Author

Thanks @jonathanmendez incorporated most of your comments, should be ready for another look over when you're ready. See 42f30e4

@bsmth
bsmth merged commit 083e44c into mozilla:main Sep 1, 2026
1 check passed
@bsmth
bsmth deleted the cta-dlp branch September 1, 2026 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants