You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by or...@apache.org on 2016/11/01 17:21:14 UTC
svn commit: r1767523 - in /qpid/java/trunk:
broker-core/src/main/java/org/apache/qpid/server/model/
broker-plugins/management-http/src/main/java/resources/css/
broker-plugins/management-http/src/main/java/resources/js/qpid/management/authenticationprov...
Author: orudyy
Date: Tue Nov 1 17:21:14 2016
New Revision: 1767523
URL: http://svn.apache.org/viewvc?rev=1767523&view=rev
Log:
QPID-7485: [Java Broker] Make the path attribute immutable on file based authentication providers
Modified:
qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/model/ExternalFileBasedAuthenticationManager.java
qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/css/common.css
qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/authenticationprovider/base64md5passwordfile/add.js
qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/authenticationprovider/plainpasswordfile/add.js
qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/query/WhereCriteria.html
qpid/java/trunk/systests/src/test/java/org/apache/qpid/systest/rest/AuthenticationProviderRestTest.java
qpid/java/trunk/systests/src/test/java/org/apache/qpid/systest/rest/acl/BrokerACLTest.java
Modified: qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/model/ExternalFileBasedAuthenticationManager.java
URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/model/ExternalFileBasedAuthenticationManager.java?rev=1767523&r1=1767522&r2=1767523&view=diff
==============================================================================
--- qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/model/ExternalFileBasedAuthenticationManager.java (original)
+++ qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/model/ExternalFileBasedAuthenticationManager.java Tue Nov 1 17:21:14 2016
@@ -24,7 +24,7 @@ public interface ExternalFileBasedAuthen
{
String PATH = "path";
- @ManagedAttribute( mandatory = true, description = "File location")
+ @ManagedAttribute( mandatory = true, description = "File location", immutable = true)
public String getPath();
Modified: qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/css/common.css
URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/css/common.css?rev=1767523&r1=1767522&r2=1767523&view=diff
==============================================================================
--- qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/css/common.css (original)
+++ qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/css/common.css Tue Nov 1 17:21:14 2016
@@ -428,12 +428,13 @@ div .messages {
width:100%;
}
-.criteriaControl.dijitTextBoxReadOnly,
-.claro .criteriaControl.dijitTextBoxReadOnly,
-.claro .criteriaControl.dijitTextBoxReadOnlyFocused .dijitInputContainer
+.readOnly.dijitTextBoxReadOnly,
+.claro .readOnly.dijitTextBoxReadOnly,
+.claro .readOnly.dijitTextBoxReadOnlyFocused .dijitInputContainer
{
background-color: #efefef !important;
background-image: none !important;
+ border-color: #d3d3d3;
}
.advancedSearchItem
Modified: qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/authenticationprovider/base64md5passwordfile/add.js
URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/authenticationprovider/base64md5passwordfile/add.js?rev=1767523&r1=1767522&r2=1767523&view=diff
==============================================================================
--- qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/authenticationprovider/base64md5passwordfile/add.js (original)
+++ qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/authenticationprovider/base64md5passwordfile/add.js Tue Nov 1 17:21:14 2016
@@ -18,17 +18,24 @@
* under the License.
*
*/
-define(["dojo/query", "dijit/registry", "qpid/common/util"], function (query, registry, util)
+define(["dojo/query", "dijit/registry", "qpid/common/util", "dojo/dom-class"], function (query, registry, util, domClass)
{
return {
show: function (data)
{
util.parseHtmlIntoDiv(data.containerNode, "authenticationprovider/filebased/add.html", function ()
{
- if (data.data)
+ var path = registry.byNode(query(".path", data.containerNode)[0]);
+ if (data.data && data.data.id)
{
- var path = registry.byNode(query(".path", data.containerNode)[0]);
path.set("value", data.data.path);
+ path.set("readOnly", true);
+ domClass.add(path.domNode, "readOnly")
+ }
+ else
+ {
+ path.set("readOnly", false);
+ domClass.remove(path.domNode, "readOnly")
}
});
}
Modified: qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/authenticationprovider/plainpasswordfile/add.js
URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/authenticationprovider/plainpasswordfile/add.js?rev=1767523&r1=1767522&r2=1767523&view=diff
==============================================================================
--- qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/authenticationprovider/plainpasswordfile/add.js (original)
+++ qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/authenticationprovider/plainpasswordfile/add.js Tue Nov 1 17:21:14 2016
@@ -18,17 +18,24 @@
* under the License.
*
*/
-define(["dojo/query", "dijit/registry", "qpid/common/util"], function (query, registry, util)
+define(["dojo/query", "dijit/registry", "qpid/common/util", "dojo/dom-class"], function (query, registry, util, domClass)
{
return {
show: function (data)
{
util.parseHtmlIntoDiv(data.containerNode, "authenticationprovider/filebased/add.html", function ()
{
- if (data.data)
+ var path = registry.byNode(query(".path", data.containerNode)[0]);
+ if (data.data && data.data.id)
{
- var path = registry.byNode(query(".path", data.containerNode)[0]);
path.set("value", data.data.path);
+ path.set("readOnly", true);
+ domClass.add(path.domNode, "readOnly")
+ }
+ else
+ {
+ path.set("readOnly", false);
+ domClass.remove(path.domNode, "readOnly")
}
});
}
Modified: qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/query/WhereCriteria.html
URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/query/WhereCriteria.html?rev=1767523&r1=1767522&r2=1767523&view=diff
==============================================================================
--- qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/query/WhereCriteria.html (original)
+++ qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/query/WhereCriteria.html Tue Nov 1 17:21:14 2016
@@ -30,7 +30,7 @@
return false;
</script>
- <input class="criteriaControl"
+ <input class="criteriaControl readOnly"
data-dojo-type="dijit/form/TextBox"
data-dojo-attach-point="newColumnCondition"
data-dojo-props="readOnly:true, title: 'Current where expression'"/>
Modified: qpid/java/trunk/systests/src/test/java/org/apache/qpid/systest/rest/AuthenticationProviderRestTest.java
URL: http://svn.apache.org/viewvc/qpid/java/trunk/systests/src/test/java/org/apache/qpid/systest/rest/AuthenticationProviderRestTest.java?rev=1767523&r1=1767522&r2=1767523&view=diff
==============================================================================
--- qpid/java/trunk/systests/src/test/java/org/apache/qpid/systest/rest/AuthenticationProviderRestTest.java (original)
+++ qpid/java/trunk/systests/src/test/java/org/apache/qpid/systest/rest/AuthenticationProviderRestTest.java Tue Nov 1 17:21:14 2016
@@ -262,11 +262,12 @@ public class AuthenticationProviderRestT
try
{
principalDatabase = getDefaultBrokerConfiguration().createTemporaryPasswordFile(new String[]{"admin2", "guest2", "test2"});
- attributes = new HashMap<String, Object>();
+ attributes = new HashMap<>();
attributes.put(AuthenticationProvider.NAME, providerName);
attributes.put(AuthenticationProvider.ID, id);
attributes.put(AuthenticationProvider.TYPE, PlainPasswordDatabaseAuthenticationManager.PROVIDER_TYPE);
- attributes.put(ExternalFileBasedAuthenticationManager.PATH, principalDatabase.getAbsolutePath());
+
+ file.createNewFile();
int status = getRestTestHelper().submitRequest("authenticationprovider/" + providerName, "PUT", attributes);
assertEquals("ACL was not deleted", 200, status);
@@ -274,7 +275,7 @@ public class AuthenticationProviderRestT
provider = getRestTestHelper().getJsonAsSingletonList("authenticationprovider/" + providerName);
assertEquals("Unexpected id", id.toString(), provider.get(AuthenticationProvider.ID));
assertEquals("Unexpected name", providerName, provider.get(AuthenticationProvider.NAME));
- assertEquals("Unexpected path", principalDatabase.getAbsolutePath() , provider.get(
+ assertEquals("Unexpected path", file.getAbsolutePath() , provider.get(
ExternalFileBasedAuthenticationManager.PATH));
assertEquals("Unexpected state", State.ACTIVE.name() , provider.get(AuthenticationProvider.STATE));
}
Modified: qpid/java/trunk/systests/src/test/java/org/apache/qpid/systest/rest/acl/BrokerACLTest.java
URL: http://svn.apache.org/viewvc/qpid/java/trunk/systests/src/test/java/org/apache/qpid/systest/rest/acl/BrokerACLTest.java?rev=1767523&r1=1767522&r2=1767523&view=diff
==============================================================================
--- qpid/java/trunk/systests/src/test/java/org/apache/qpid/systest/rest/acl/BrokerACLTest.java (original)
+++ qpid/java/trunk/systests/src/test/java/org/apache/qpid/systest/rest/acl/BrokerACLTest.java Tue Nov 1 17:21:14 2016
@@ -43,6 +43,7 @@ import org.apache.qpid.server.model.Grou
import org.apache.qpid.server.model.KeyStore;
import org.apache.qpid.server.model.Port;
import org.apache.qpid.server.model.Protocol;
+import org.apache.qpid.server.model.State;
import org.apache.qpid.server.model.TrustStore;
import org.apache.qpid.server.model.adapter.FileBasedGroupProvider;
import org.apache.qpid.server.model.adapter.FileBasedGroupProviderImpl;
@@ -154,13 +155,10 @@ public class BrokerACLTest extends QpidR
assertAuthenticationProviderExists(providerName);
- File file = TestFileUtils.createTempFile(this, ".users", "guest:guest\n" + ALLOWED_USER + ":" + ALLOWED_USER + "\n"
- + DENIED_USER + ":" + DENIED_USER);
-
- Map<String, Object> attributes = new HashMap<String, Object>();
+ Map<String, Object> attributes = new HashMap<>();
attributes.put(AuthenticationProvider.NAME, providerName);
attributes.put(AuthenticationProvider.TYPE, PlainPasswordDatabaseAuthenticationManager.PROVIDER_TYPE);
- attributes.put(ExternalFileBasedAuthenticationManager.PATH, file.getAbsolutePath());
+ attributes.put(AuthenticationProvider.STATE, State.DELETED.name());
int responseCode = getRestTestHelper().submitRequest("authenticationprovider/" + providerName, "PUT", attributes);
assertEquals("Setting of provider attribites should be allowed", 200, responseCode);
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org