ikiwiki/doc/todo/git_attribution/discussion.mdwn

66 lines
2.6 KiB
Markdown

I'd strongly recommend this modification to ikiwiki. Any particular limitations that anyone can think of?
I might even have a try at this patch, though I'd have to hack the user preferences page to include author name...
As to the question of whether the committer was the 'script' or the wiki editor... I'm not sure. Marking it as the script somehow (`ikiwiki-cgi <ikiwiki@sitename>`)? seems to make sense and would make it easier to manage.
[[harningt]]
I've been thinking a bit about the GIT attribution in ikiwiki...
If no email set, I think "$USERNAME" is reasonable... no point in the
'<>' causing clutter.
>> **adjustement wrt comments**: leave the '<>' in due to requirements in git
If no username set... then something like '@[IPADDR]' makes sense...
(not in email brackets).
> Why not put it in email brackets? --[[Joey]]
In the case of OpenID login.. I think that's a special case... I don't
think attempting to munge something meaningful out of the OpenID makes
sense... but I think some massaging might need to be done.
Ex: I've noticed in the current mode where logging in w/
harningt.eharning.us/ shows up in the logs w/o HTTP and if I login w/
http://harningt.eharning.us/ is shows up w/ the http... causing some
inconsistency. I think it oughtta make sure that it has the properly
discovered, canonicalized form (ex: if there's a redirect to another
site (harningt.eharning.us -> www.eharning.us) then technically the
target site is the 'real' openid (at least according to how most OpenID
RPs take it).
...
For OpenID edits, I think there should be a way to tell it what
username to show in the preferences dialog (so you can have a 'normal'
$USER <$EMAIL> setup.) This could by default be filled in w/ sreg
nickname value (as well as email for that matter)...
To convey the openid used to make the edit, I think it would be
important that some sort of footer line along the lines of the
Signed-off: $USER <$EMAIL> conventions I've seen.
Perhaps an OpenID: $OPENID_URL would make sense. This could help w/
making sure that no one irrefutably spoofs a post by someone (since w/
the setup where email and effective username are configurable, there's
no determination of uniqueness)
>> **adj re git req**: "$OPENID_URL <>"
[[harningt]]
[[madduck]]: git requires `Name <Email@address>` format, as far as I know.
> Yes, it does:
>
> joey@kodama:~/tmp/foo/bar>git commit --author "foo"
> fatal: malformed --author parameter
>
> It seems to be happy with anything of the form "foo <foo>" -- doesn't seem to
> do any kind of strict checking. Even "http://joey.kitenet.net <>" will be
> accepted. --[[Joey]]
>>
>>Sounds good to me,
>>
>> --[[harningt]]