Re: [PATCH] gitweb: redacted e-mail addresses feature.
flat view
From: Georgios Kontaxis <hidden>
Date: 2021-03-21 07:06:55
"Georgios Kontaxis via GitGitGadget" [off-list ref] writes:quoted
+ # To disable system wide have in $GITWEB_CONFIG + # $feature{'email_privacy'}{'default'} = [0]; + 'email_privacy' => { + 'sub' => sub { feature_bool('email_privacy', @_) }, + 'override' => 0, + 'default' => [1]}, );I do not see why this should default to true.
I've changed the default to "false". V2 should reflect the change.
It would break existing installations, who have been perfectly happy with the convenience of supplying a ready access to potential new contributors who/which addresses to contact plausible mentors in the projects they are interested in. And more importantly, I do not see why it should be made impossible to override per repository/project in a multi-tenant installation. Some projects may be more "privacy" sensitive than others. Those who want to use tighter setting should be able to enable it even when the side-wide default is set to false, no? Thanks.
I was actually thinking about the other way around; preventing projects from disabling this feature. Sounds like the "override" flag is for other types of use cases though. I'll change it to "true". Thanks for the feedback.