[chc]

AFAIK this is not the case. The contributor had not signed a CLA when he first submitted the pull request, but evidently did sign it well before Isaac merged in the change (he claimed to have done so very early in the original PR thread). As far as I can tell, procedure was followed.

It seems pretty clear that Ben acted somewhat inappropriately — maybe not with his initial rejection, but when he reverted Isaac's merge and "chided" him. The important thing to remember, though, is that one bit of prickly behavior shouldn't be enough to brand someone "an asshole." Heaven knows I've gotten annoyed and said things I've regretted on more than one occasion.

[Lazare]

Well, I admit to not being 100% clear on the CLA. I was going of comments in one of the other threads from someone who seemed to know what they're talking about, but didn't show proof.

Still though, I would make the reverse point: I think the initial rejection was a bit dodgy, but the reversion was clearly justified.

Really, the reversion had nothing to do with the CLA; rather it was made by Isaac, who did NOT have authority to make that commit, and as far as Ben knew, it was unauthorized. Reverting it was justified. (In fact it was authorized by Bert, but Ben didn't know that. Yeah, Ben could have checked with Bert first, but I doubt I'd have done that in his shoes. If I was making a commit to a project I didn't have authority to, I'd probably make a public note who had told me it was okay. Isaac didn't.)

...still, either way, I agree with your conclusion.