Change in ovirt-engine[master]: common: handle resources if path contains special characters

dfediuck at redhat.com dfediuck at redhat.com
Thu Aug 30 15:26:05 UTC 2012


Doron Fediuck has submitted this change and it was merged.

Change subject: common: handle resources if path contains special characters
......................................................................


common: handle resources if path contains special characters

ClassLoader.getResource() returns URL, URL.getPath() returns URI
encoded string. Trying to use this string to access file will fail
without decoding it first.

Change-Id: Iff53340e6243b9e7c634ba00f30bac685b82ef80
Signed-off-by: Alon Bar-Lev <alonbl at redhat.com>
---
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dal/dbbroker/generic/DomainsPasswordMapTest.java
M backend/manager/modules/restapi/interface/common/jaxrs/src/main/java/org/ovirt/engine/api/common/util/PackageExplorer.java
M backend/manager/modules/restapi/interface/common/jaxrs/src/main/java/org/ovirt/engine/api/common/util/ReflectionHelper.java
M backend/manager/tools/engine-config/src/test/java/org/ovirt/engine/core/config/EngineConfigTest.java
4 files changed, 14 insertions(+), 8 deletions(-)

Approvals:
  Doron Fediuck: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/7497
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iff53340e6243b9e7c634ba00f30bac685b82ef80
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list