Konstantin Ryabitsev <konstan...@linuxfoundation.org> wrote: > Hi, all: > > What do you think about a mechanism to run hooks at the stage right before > public-inbox-watch adds a new message to the archive? One feature that would > be neat is to search archives for all instances of the same patch using its > $(git-patch-id --stable) and adding a header, e.g.: > > X-Git-Patch-ID: 19c05284cea20b72b44c2b7e6cfd782a6a860cf1 > 0000000000000000000000000000000000000000
There shouldn't be any need for a header. I've been meaning to teach -index to use git-patch-id and index it's output, anyways. That would work for old messages, too. > I know this can be done at the postfix stage, but seems like it would be more > efficient at the ingestion stage. > > Maybe even instead of a hook this could be a native public-inbox feature, with > this header being indexed by default? I've always tried as hard as possible to avoid adding new headers or extra data; especially when we have a common and stable tool to generate it. Another danger is a malicious client could also be introducing wrong ones to confuse searches. Thanks.