Re: Deadname rewriting

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jun 17 2019, Philip Oakley wrote:

> On 15/06/2019 09:19, Ævar Arnfjörð Bjarmason wrote:
>> On Sat, Jun 15 2019, Phil Hord wrote:
>>
>>> I know name-scrubbing is already covered in filter-branch and other
>>> places. But we have a scenario becoming more common that makes it a
>>> more sensitive topic.
>>>
>>> At $work we have a long time employee who has changed their name from
>>> Alice to Bob.  Bob doesn't want anyone to call him "Alice" anymore and
>>> is prone to be offended if they do.  This is called "deadnaming".
>>>
>>> We are able to convince most of our work tools to expunge the deadname
>>> from usage anywhere, but git stubbornly calls Bob "Alice" whenever
>>> someone asks for "git blame" or checks in "git log".
>>>
>>> We could rewrite history with filter-branch, but that's quite
>>> disruptive.  I found some alternatives.
>>>
>>> .mailmap seems perfect for this task, but it doesn't work everywhere
>>> (blame, log, etc.).  Also, it requires the deadname to be forever
>>> proclaimed in the .mailmap file itself.
>>>
>>> `git replace` works rather nicely, except all of Bob's old commits
>>> show "replaced" in the decorator list. Also, it doesn't propagate well
>>> from the central server since `refs/replaces` namespace isn't fetched
>>> normally.  But in case anyone wants it, here's what I did:
>>>
>>> git log --author=alice.smith --format="%h" --all |
>>>     while read hash ; do
>>>        GIT_EDITOR='sed -i -e s/Alice Smith/Bob Smith/g' -e
>>> 's/alice.smith/bob.smith/' \
>>>        git replace --edit $hash
>>>     done
>>> git push origin 'refs/replace/*:refs/replace/*'
>>>
>>> I'd quite like the .mailmap solution to work, and I might flesh that
>>> out that some day.
>>>
>>> It feels like `.git/info/grafts` would work the best if it could be
>>> distributed with the project, but I'm pretty sure that's a non-starter
>>> for many reasons.
>>>
>>> Any other ideas?  Has anyone here encountered this already?
>> What should be done is to extend the .mailmap support to other
>> cases. I.e. make tools like blame, shortlog etc. show the equivalent of
>> %aN and %aE by default.
>>
>> This topic was discussed at the last git contributor summit (brought up
>> by CB Bailey) resulting in this patch, which I see didn't make it in &
>> needs to be resurrected again:
>> https://public-inbox.org/git/20181212171052.13415-1-cb@xxxxxxxxxxxxx/
>>
>> So, patches welcome :)
>>
>> What's not going to be supported is some notion of 100% forgetting that
>> there was ever an Alice that's now called Bob. They did in fact create
>> commit objects with "Alice" in them, and low-level plumbing like
>> "cat-file -p <commit>" is always going to show that, and there's going
>> to be the mapping in .mailmap.
>>
>> But as far as porcelain UI things that would show the mailmapped value
>> goes those can be made to always show "Bob".
>>
>> Unless of course your $work is willing to completely rewrite the repo...
> This may become a bigger issue for corporates that prevents Git from
> being used because it doesn't handle the _legal requirements_ for
> proper current `known-by:` naming.
>
> I found this [1] on the UK Parliament website that also covers
> 'deadnaming', and the potential misunderstandings about what is (and
> is not) a (unnecessary) 'legal name'.
>
> It may be an option for the SHA1 transition to also include, as an
> independent step, the appropriate mailmap conversion for dead-names
> (which is a private document owned by the hosting repo owner - see
> GDPR Data Controller responsibilities).
>
> If author/committer renaming is done as part of a full hash conversion
> (with a golden repo providing hash mapping) then it is less of a
> problem for a one-shot conversion, but still an issue for everyday
> name changes (including those from divorce, adoption, etc). Maybe even
> convert (swap) the ascii/utf-8 names for unique hashes (in the repo)
> for reverse look up of the latest known-by name (getting a bit
> complicated here)
>
> The distributed nature of the classic Git open source usage may have
> similar issues to that of gmane, where it pulled the hosting of email
> lists. A legal case is likely needed before any level of clarification
> is obtained (which will still have overlaps!)
>
> The mailmap is probably not the right place for holding deadname
> conversions as they should not be public, but it may be a partial
> workaround to reduce visibility of deadnames.
>
> [1]
> https://publications.parliament.uk/pa/cm201516/cmselect/cmwomeq/390/39009.htm

I don't see how tacking this onto the SHA-1->SHA-256 hash transition
could ever work.

Ignoring the issues with how it wouldn't work with the current plan as
designed, you'd have a mostly 1=1 mapping between the two hashes, except
in cases where commits from "Alice" wouldn't 1=1 map, because they'd
been subject to some mailmap munging you didn't have access to to map
them to "Bob".

I'd think that in the context of a non-public work repository you'd just
leak the same information anyway. People wanting to deadname themselves
is relatively rare, so it wouldn't be hard in most cases to infer the
missing data, and you'd be back at square 1.

That, and even if it somehow worked the hash transition is a one-off
thing, and you'd presumably want this on an ongoing basis, so you'd need
some other mechanism.

I think there's certainly a place for better support for gracefully
handling complete history rewriting in a centralized workflow better
within git.

That sort of thing is useful for other things, e.g. you might want to
rewrite out some old big blob in your history, or erase any record that
you used to use an indenting style that's not in fashion.

So basically an improvement to the refs/replaces facility where clients
would opt-in to eagerly forget the old history, and since it your $work
you'd have a central server you could enforce the eventual full
transition, and clients wouldn't get upset about the non-fast-forward.




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux