On Tue, 3 Jan 2023 at 12:49, 'Jesse Glick' via Jenkins Developers
<jenkinsci-dev@googlegroups.com> wrote:
>
> On Fri, Dec 30, 2022 at 8:17 AM Cristian Morales Vega <crist...@samknows.com> 
> wrote:
>>
>> I am looking at https://issues.jenkins.io/browse/JENKINS-70342, but
>> I'm not sure how the "context" in CanonicalIdResolver is supposed to
>> be used. The only thing I have seen in the docs is "Can be used (for
>> example) to distinguish ambiguous committer ID using the SCM URL".
>>
>> DefaultUserCanonicalIdResolver, User.FullNameIdResolver,
>> User.UserIDCanonicalIdResolver ignore it; and I have not seen it used
>> anywhere else, neither giving nor receiving a context. But I'm new
>> here, there is no easy way to see all the implementations of
>> CanonicalIdResolver in all the plugins, there is?
>
>
> https://github.com/jenkinsci/additional-identities-plugin/blob/f98f43fc8a2b1831779e48cb5355bf1295ce347f/src/main/java/com/cloudbees/jenkins/plugins/AdditionalIdentityResolver.java#L51-L62
>  seems to look for it. But it could only come from 
> https://github.com/jenkinsci/jenkins/blob/9d961dcce5f9e6094d2b9bf38ae4a112a27df4d8/core/src/main/java/hudson/model/User.java#L509-L510
>  and I do not recall any code ever passing this argument to `User.get`. I 
> suspect this aspect of the API was never really developed.

For my simple needs, the additional-identities-plugin actually solves
all my issues!


>> The use of "idOrFullName" in resolveCanonicalId() also feels a bit
>> overspecific? While you could call an email an "id", you could argue
>> the same about a full name.
>
>
> Full name would be something like “First M. Last”. It is possible to pass 
> this to `User.get` to look up a user by either id or full name. In practice 
> we try to always use `getById`. The id might be something like `flast` or 
> like `fl...@github.com` or something else, depending on the `SecurityRealm`.
>
> The entire system of associating “ids”, full names, and email addresses with 
> both `SecurityRealm` authentications and SCM commit “authors” in Jenkins is a 
> complicated mess I am afraid.

FWIW I have created https://issues.jenkins.io/browse/JENKINS-70382

-- 
You received this message because you are subscribed to the Google Groups 
"Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to jenkinsci-dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/jenkinsci-dev/CAGUfO%2ByRmFgt%2B%3DPERobKsLuDs_Utu%3D8tTv0-HVT7Qg8btorfNA%40mail.gmail.com.

Reply via email to