Skip to content

Conversation

ChaosExAnima
Copy link
Contributor

@ChaosExAnima ChaosExAnima commented May 19, 2025

Adds a setting to control whether an instance shares referrer headers, and consolidates two groups of settings.

Screenshot 2025-05-19 at 14 27 25

@ChaosExAnima ChaosExAnima added security Security issues and fixes, vulnerabilities moderation Administration and moderation tooling labels May 19, 2025
@ChaosExAnima ChaosExAnima requested a review from a team May 19, 2025 10:48
@ChaosExAnima ChaosExAnima force-pushed the echo/mas-403-add-a-setting-for-the-referer-header-in-admin-panel branch from 6edccb9 to 6cf9490 Compare May 19, 2025 10:50
Copy link
Contributor

@ClearlyClaire ClearlyClaire left a comment

Choose a reason for hiding this comment

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

I was going to say this is not about “Public timelines”, but neither is “Opt users out of search engine indexing by default”, so I'm not sure what the best thing to do would be here. Maybe yet another section, but this page already has a bunch of single-option sections.

Looks good otherwise.

@ChaosExAnima ChaosExAnima force-pushed the echo/mas-403-add-a-setting-for-the-referer-header-in-admin-panel branch from c613351 to 8f46b67 Compare May 19, 2025 11:07
ClearlyClaire
ClearlyClaire previously approved these changes May 19, 2025
@ChaosExAnima ChaosExAnima force-pushed the echo/mas-403-add-a-setting-for-the-referer-header-in-admin-panel branch from 8f46b67 to d8d0695 Compare May 19, 2025 11:46
@ChaosExAnima ChaosExAnima force-pushed the echo/mas-403-add-a-setting-for-the-referer-header-in-admin-panel branch from d8d0695 to e2d2794 Compare May 19, 2025 11:56
@Gargron Gargron added this pull request to the merge queue May 19, 2025
Merged via the queue into main with commit dbf1478 May 19, 2025
32 checks passed
@Gargron Gargron deleted the echo/mas-403-add-a-setting-for-the-referer-header-in-admin-panel branch May 19, 2025 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
moderation Administration and moderation tooling security Security issues and fixes, vulnerabilities
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants