jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From FĂ©licien FRANCOIS <notificati...@github.com>
Subject [jclouds/jclouds] Fix WindowsEveryonePrincipalName resolving when it contains spaces (#1051)
Date Tue, 10 Jan 2017 17:18:23 GMT
On french windows, the everyone principal name is "Tout le monde"

The previous code resolved it as "Tout" which was breaking the putBlob step with the following
exception:
```
java.nio.file.attribute.UserPrincipalNotFoundException: null
	at sun.nio.fs.WindowsUserPrincipals.lookup(Unknown Source)
	at sun.nio.fs.WindowsFileSystem$LookupService$1.lookupPrincipalByName(Unknown Source)
	at org.jclouds.filesystem.util.Utils.setPrivate(Utils.java:154)
	at org.jclouds.filesystem.strategy.internal.FilesystemStorageStrategyImpl.setBlobAccess(FilesystemStorageStrategyImpl.java:585)
	at org.jclouds.filesystem.strategy.internal.FilesystemStorageStrategyImpl.putBlob(FilesystemStorageStrategyImpl.java:498)
	at org.jclouds.blobstore.config.LocalBlobStore.putBlob(LocalBlobStore.java:763)
	at org.jclouds.blobstore.config.LocalBlobStore.putBlob(LocalBlobStore.java:523)
```

Splitting by 2 spaces should works on all locals and fix the issue
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Fix WindowsEveryonePrincipalName resolving when it contains spaces

-- File Changes --

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

-- Patch Links --

https://github.com/jclouds/jclouds/pull/1051.patch
https://github.com/jclouds/jclouds/pull/1051.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/1051

Mime
View raw message