Hi Fabrizio,
Obviously you figured it out beacause you had a pull request for this.
Indeed, we were only talking about the link name not the qualified name.
Andy
On Wed, 5 Apr 2017, xrootd-dev wrote:
> Hi again,
>
> I am on this small thing, and re-re-re-read the notes. They
> were mentioning only the link name for this 8-chars thing. Not
> SecEntity.name
>
> In fact I believe it would make no sense to mangle the username, would it ?
>
> f
>
>
>
>
> On 05.04.17 09:40, Fabrizio Furano wrote:
>> Ciao Gerri,
>>
>> indeed, I forgot to shorten the name. Good that you don't share the other doubts I had.
>> Thank you for the review, I hope I'll be able to do this today-ish
>>
>> f
>>
>>
>> On 04/04/2017 06:38 PM, Gerardo GANIS wrote:
>>> Hi Fabrizio,
>>>
>>> I am not 100% sure, but for XrdSecEntity.name in the case no username was specified, I thought we said to take the last 8 digits
>>> of the first '/CN=', without spaces, i.e., from '.../CN=Fabrizio Furano/...' we should have got 'ioFurano' .
>>> If that is the case, then I am not sure the code does that: the buffer is 64 in length and the space is not removed.
>>> Do I remember wrongly?
>>>
>>> Gerri
>>>
>>> ÿÿ
>>> You are receiving this because you commented.
>>> Reply to this email directly, view it on GitHub <https://github.com/xrootd/xrootd/pull/495#issuecomment-291558463>, or mute the
>>> thread <https://github.com/notifications/unsubscribe-auth/AD7YjvOJ0pnIjHWZCAze5mUFUAU88YLEks5rsnHvgaJpZM4MxvXG>.
>>>
>>>
>>> --------------------------------------------------------------------------------------------------------------------------------
>>>
>>> Use REPLY-ALL to reply to list
>>>
>>> To unsubscribe from the XROOTD-DEV list, click the following link:
>>> https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1
>>>
>
>
>
> --
> You are receiving this because you modified the open/close state.
> Reply to this email directly or view it on GitHub:
> https://github.com/xrootd/xrootd/pull/495#issuecomment-291877014
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/xrootd/xrootd","title":"xrootd/xrootd","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/xrootd/xrootd"}},"updates":{"snippets":[{"icon":"PERSON","message":"@abh3 in #495: Hi Fabrizio,\n\nObviously you figured it out beacause you had a pull request for this. \nIndeed, we were only talking about the link name not the qualified name.\n\nAndy\n\nOn Wed, 5 Apr 2017, xrootd-dev wrote:\n\n\u003e Hi again,\n\u003e\n\u003e I am on this small thing, and re-re-re-read the notes. They\n\u003e were mentioning only the link name for this 8-chars thing. Not\n\u003e SecEntity.name\n\u003e\n\u003e In fact I believe it would make no sense to mangle the username, would it ?\n\u003e\n\u003e f\n\u003e\n\u003e\n\u003e\n\u003e\n\u003e On 05.04.17 09:40, Fabrizio Furano wrote:\n\u003e\u003e Ciao Gerri,\n\u003e\u003e\n\u003e\u003e indeed, I forgot to shorten the name. Good that you don't share the other doubts I had.\n\u003e\u003e Thank you for the review, I hope I'll be able to do this today-ish\n\u003e\u003e\n\u003e\u003e f\n\u003e\u003e\n\u003e\u003e\n\u003e\u003e On 04/04/2017 06:38 PM, Gerardo GANIS wrote:\n\u003e\u003e\u003e Hi Fabrizio,\n\u003e\u003e\u003e\n\u003e\u003e\u003e I am not 100% sure, but for XrdSecEntity.name in the case no username was specified, I thought we said to take the last 8 digits\n\u003e\u003e\u003e of the first '/CN=', without spaces, i.e., from '.../CN=Fabrizio Furano/...' we should have got 'ioFurano' .\n\u003e\u003e\u003e If that is the case, then I am not sure the code does that: the buffer is 64 in length and the space is not removed.\n\u003e\u003e\u003e Do I remember wrongly?\n\u003e\u003e\u003e\n\u003e\u003e\u003e Gerri\n\u003e\u003e\u003e\n\u003e\u003e\u003e ÿÿ\n\u003e\u003e\u003e You are receiving this because you commented.\n\u003e\u003e\u003e Reply to this email directly, view it on GitHub \u003chttps://github.com/xrootd/xrootd/pull/495#issuecomment-291558463\u003e, or mute the\n\u003e\u003e\u003e thread \u003chttps://github.com/notifications/unsubscribe-auth/AD7YjvOJ0pnIjHWZCAze5mUFUAU88YLEks5rsnHvgaJpZM4MxvXG\u003e.\n\u003e\u003e\u003e\n\u003e\u003e\u003e\n\u003e\u003e\u003e --------------------------------------------------------------------------------------------------------------------------------\n\u003e\u003e\u003e\n\u003e\u003e\u003e Use REPLY-ALL to reply to list\n\u003e\u003e\u003e\n\u003e\u003e\u003e To unsubscribe from the XROOTD-DEV list, click the following link:\n\u003e\u003e\u003e https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV\u0026A=1\n\u003e\u003e\u003e\n\u003e\n\u003e\n\u003e\n\u003e -- \n\u003e You are receiving this because you modified the open/close state.\n\u003e Reply to this email directly or view it on GitHub:\n\u003e https://github.com/xrootd/xrootd/pull/495#issuecomment-291877014"}],"action":{"name":"View Pull Request","url":"https://github.com/xrootd/xrootd/pull/495#issuecomment-292003264"}}}
Use REPLY-ALL to reply to list
To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1