You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by mildis <no...@github.com> on 2017/01/05 08:16:58 UTC

[jclouds/jclouds] WindowsEveryonePrincipal : update regex for FR (#1049)

"Everyone" in a french Windows is "Tout le monde".
Thus, the line.split(" ") returns only "Tout" and putBlob() throws an exception.
The modified regex search for the first two consecutive white char to return the name.
Windows's whoami returns a bunch of 0x20 between the name and the type.
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds/pull/1049

-- Commit Summary --

  * WindowsEveryonePrincipal : update regex for FR

-- File Changes --

    M apis/filesystem/src/main/java/org/jclouds/filesystem/util/Utils.java (2)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/1049.patch
https://github.com/jclouds/jclouds/pull/1049.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1049

Re: [jclouds/jclouds] WindowsEveryonePrincipal : update regex for FR (#1049)

Posted by Zack Shoylev <no...@github.com>.
merged and backported to 2.0.x

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1049#issuecomment-271684603

Re: [jclouds/jclouds] WindowsEveryonePrincipal : update regex for FR (#1049)

Posted by Zack Shoylev <no...@github.com>.
Closed #1049.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1049#event-918104364

Re: [jclouds/jclouds] WindowsEveryonePrincipal : update regex for FR (#1049)

Posted by Zack Shoylev <no...@github.com>.
Ah, very good find.

+1

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1049#issuecomment-270681381

Re: [jclouds/jclouds] WindowsEveryonePrincipal : update regex for FR (#1049)

Posted by mildis <no...@github.com>.
Created https://issues.apache.org/jira/browse/JCLOUDS-1222 to match this PR.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1049#issuecomment-270947547

Re: [jclouds/jclouds] WindowsEveryonePrincipal : update regex for FR (#1049)

Posted by mildis <no...@github.com>.
@zack-shoylev Hope it could be included in the next 2.0.x fix release.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1049#issuecomment-270687242