13 Dec, 2013
6 commits
-
Most of the replace was done by global search replace from Eclipse via regular expressions. Individual files had incorrect headers or badly formatted ones, so some of them have been done manually by replacing the content between CDDL HEADER START and CDDL HEADER END. Some headers were quoting "trunk/opends/resource/legal-notices/CDDLv1_0.txt" while it should have been "legal-notices/CDDLv1_0.txt". Removed: - resource/legal-notices/OpenDS.LICENSE The following required manual changes: - build.xml - ext/checkstyle/opendj.sourceheader - resource/admin/java-utilities.xsl - src/build-tools/org/opends/build/tools/ProcessFilesForPackages.java - src/build-tools/windows/* - src/pkg/opends-dsml-gateway_proto.py - src/pkg/opends_proto.py - src/server/org/opends/server/util/ServerConstants.java - tests/staf-tests/functional-tests/shared/data/aci/aci_targattrfilter/add_aci18.ldif - tests/staf-tests/build.bat An incorrect header mentionning "exclude" instead of "include" in the copyright header existed for: tests/staf-tests/functional-tests/testcases/backends/backup/**/* Some files have no copyright headers at all. I did not update them. Here is some of them (more exist): - tests/staf-tests/functional-tests/shared/data/core/psearch/**/*.ref - tests/staf-tests/functional-tests/shared/data/core/psearch/**/delete.ldif git-svn-id: https://svn.forgerock.org/opendj/trunk@9984 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
* Migrate and fix all unit tests in org.opends.server.admin package ** 18 test cases, 409 tests ** Remove server start in setup method of all test cases ** Test case ValidateConfigDefinitionsTest is disabled for now as it needs schema with config object classes * Update pom.xml ** Change location of generated messages for managed objects to src/main/resources directory * Update DirectoryServer class : comments and implementation of some stub methods * Reformat some classes in opends.server.admin package git-svn-id: https://svn.forgerock.org/opendj/trunk@9983 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
Most of the replace was done by global search replace from Eclipse via regular expressions. Individual files had incorrect headers or badly formatted ones, so some of them have been done manually by replacing the content between CDDL HEADER START and CDDL HEADER END. Some headers were quoting "trunk/opends/resource/legal-notices/CDDLv1_0.txt" while it should have been "legal-notices/CDDLv1_0.txt". Removed: - resource/legal-notices/OpenDS.LICENSE The following required manual changes: - build.xml - ext/checkstyle/opendj.sourceheader - resource/admin/java-utilities.xsl - src/build-tools/org/opends/build/tools/ProcessFilesForPackages.java - src/build-tools/windows/* - src/pkg/opends-dsml-gateway_proto.py - src/pkg/opends_proto.py - src/server/org/opends/server/util/ServerConstants.java - tests/staf-tests/functional-tests/shared/data/aci/aci_targattrfilter/add_aci18.ldif - tests/staf-tests/build.bat An incorrect header mentionning "exclude" instead of "include" in the copyright header existed for: tests/staf-tests/functional-tests/testcases/backends/backup/**/* Some files have no copyright headers at all. I did not update them. Here is some of them (more exist): - tests/staf-tests/functional-tests/shared/data/core/psearch/**/*.ref - tests/staf-tests/functional-tests/shared/data/core/psearch/**/delete.ldif git-svn-id: https://svn.forgerock.org/opendj/trunk@9982 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
…o TRUE which causes an error in post ugrade task (rebuild-index) git-svn-id: https://svn.forgerock.org/opendj/trunk@9981 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
In testGenerateDoc(), added the @Test annotation + ensured the property set is cleared at the end of the test. git-svn-id: https://svn.forgerock.org/opendj/trunk@9980 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9979 41b1ffd8-f28e-4786-ab96-9950f0a78031
12 Dec, 2013
9 commits
-
* fail the connection when a bind or startTLS request times out * ensure that abandon requests are sent for other types of operation * fix a bug which meant that the timeout checker could fail to enforce timeouts for the first LDAP connection * minor improvements: only register connections which have a non-zero timeout; remove duplicate requestID field from LDAP future implementation * added unit tests. These tests are a bit too functional at the moment and will be split up in a subsequent change. git-svn-id: https://svn.forgerock.org/opendj/trunk@9975 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
…ry persistent search requests * don't enforce timeouts for searches that contain the control 1.2.840.113556.1.4.528 git-svn-id: https://svn.forgerock.org/opendj/trunk@9974 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9973 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
VirtualAttributeProvider.java: Pulled up the empty implementations of initializeVirtualAttributeProvider(). Fixed typos. *.java: Consequence of pulling up empty implementations of initializeVirtualAttributeProvider(). Automatic + manual code cleanups. git-svn-id: https://svn.forgerock.org/opendj/trunk@9972 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
See the test failure stack trace at the bottom of this commit message. ECLReplicationServerPreTest(), the very first test of ExternalChangeLogTest, expects external changelog to be disabled. However, when GroupIdHandshakeTest was run before ExternalChangeLogTest but never removed ECL virtual attributes. This led to the "lastChangeNumber" virtual attribute having an actual value of "0" while it was expected it would not exist on the returned entry. Moved the total responsibility of maintaining the virtual attributes to VirtualAttributeConfigManager. Ensured the ECL virtual attributes are removed on ECL shutdown. VirtualAttributeConfigManager.java: Do not call anymore the DirectoryServer.(de)registerVirtualAttribute() methods. Added getVirtualAttributes(), register(), deregister(), getDummyDN(). In loadProvider(), removed the use of reflection. DirectoryServer.java: Removed virtualAttributes field. Delegated getVirtualAttributes(), registerVirtualAttribute(), deregisterVirtualAttribute() calls to VirtualAttributeConfigManager. Removed replaceVirtualAttribute(). ReplicationServer.java: Renamed externalChangeLogWorkflowID field to eclWorkflowID and externalChangeLogWorkflowImpl field to eclWorkflowImpl. Changed eclWorkflowImpl to use an AtomicReference. In enableECL(), used AotmicReference.compareAndSet() to avoid multiple threads initializing at the same time. Renamed enableECLVirtualAttr() to buildVirtualAttributeRule() + extracted the call to DirectoryServer.registerVirtualAttribute() outside. In buildVirtualAttributeRule(), used Collections.* methods. In shutdownECL(), deregister the virtual attributes added by the enableECL(). [testng] T E S T F A I L U R E ! ! ! [testng] [testng] Failed Test: org.opends.server.replication.server.ExternalChangeLogTest#ECLReplicationServerPreTest [testng] Failure Cause: java.lang.AssertionError: expected:<null> but was:<0> [testng] org.testng.Assert.fail(Assert.java:84) [testng] org.testng.Assert.failNotEquals(Assert.java:438) [testng] org.testng.Assert.assertEquals(Assert.java:108) [testng] org.testng.Assert.assertEquals(Assert.java:129) [testng] org.testng.Assert.assertEquals(Assert.java:139) [testng] org.opends.server.replication.server.ExternalChangeLogTest.ECLCompatTestLimits(ExternalChangeLogTest.java:2560) [testng] org.opends.server.replication.server.ExternalChangeLogTest.ECLIsNotASupportedSuffix(ExternalChangeLogTest.java:407) [testng] org.opends.server.replication.server.ExternalChangeLogTest.ECLReplicationServerPreTest(ExternalChangeLogTest.java:167) git-svn-id: https://svn.forgerock.org/opendj/trunk@9971 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
Code review: Matthew Swift build.xml: Removed special checks checkstyle for org.opends.server.api, org.opends.server.protocols.internal, org.opends.server.types packages. Matt said they were added to ease embedding code into the docs, but this never happened, so this feature is basically dead. Extracted property named "buildtools.jar" opends-doctarget-checkstyle.xml: REMOVED git-svn-id: https://svn.forgerock.org/opendj/trunk@9970 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
* Add ConfigurationMock class to enable creation of Configuration objects mocks with default values automatically enabled ** Add ConfigurationMockTest to test correct behavior of mocking * Add default implementation of some DirectoryServer class methods to enable tests ** Use default SDK schema * Add dependency on opendj-core test jar in pom.xml git-svn-id: https://svn.forgerock.org/opendj/trunk@9969 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
Code cleanup git-svn-id: https://svn.forgerock.org/opendj/trunk@9968 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
Extracted methods createRule(), buildFilters(), areFiltersAcceptable(). In loadProvider(), called StaticUtils.collectionToString(). git-svn-id: https://svn.forgerock.org/opendj/trunk@9967 41b1ffd8-f28e-4786-ab96-9950f0a78031
11 Dec, 2013
5 commits
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9962 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9961 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
Extracted method hasGroupMembershipUpdateControl(). In loadGroup(), used StaticUtils.collectionToString(). Automatic code cleanups. git-svn-id: https://svn.forgerock.org/opendj/trunk@9960 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
SetPropSubCommandHandler.java: Extracted methods parseValue() and getPropertyDefinition(). Argument.java: Improved the toString() + javadocs. Minor code cleanups. ArgumentParser.java: Removed one parseArguments() method - never used. Extracted method indentAndWrap() to match SubCommandArgumentParser. Minor code cleanups. SubCommandArgumentParser.java: Removed one parseArguments() method - never used. Minor code cleanups. DsconfigOptionsTestCase.java: Added testGenerateDoc(). Minor code cleanups. git-svn-id: https://svn.forgerock.org/opendj/trunk@9959 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9958 41b1ffd8-f28e-4786-ab96-9950f0a78031
10 Dec, 2013
6 commits
-
* ensure that temporary enviroment is always closed. git-svn-id: https://svn.forgerock.org/opendj/trunk@9955 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9954 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9953 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
Remove all remaining compilation errors in org.opends.server.admin packages * Replace client.AuthorizationException and client.CommunicateException classes by ErrorResultException from SDK ** Update stylesheets clientMO.xsl and metaMO.xsl to use ErrorResultException in generated code * Replace all javax.naming classes by SDK classes ** Remove JNDIDirContextAdaptor class * Replace all remaining occurences of RDN, DN and Attribute classes from server by SDK classes * Other minor changes git-svn-id: https://svn.forgerock.org/opendj/trunk@9952 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
Implemented toString(). Code cleanup. git-svn-id: https://svn.forgerock.org/opendj/trunk@9951 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9950 41b1ffd8-f28e-4786-ab96-9950f0a78031
09 Dec, 2013
12 commits
-
…ach import. This shouldn't be necessary since there is only one suffix per backend. git-svn-id: https://svn.forgerock.org/opendj/trunk@9949 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9948 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9947 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9946 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9945 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9944 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
Code review: Matthew Swift Regression caused by the fix for OPENDJ-948 unauthorized disclosure of directory contents. The code was looking for the entry matching the DN we are adding which obviously was null here. Since it was null, the code went on to wrongly check the following conditions: op.isInternalOperation() || op.isSynchronizationOperation() || op.isInnerOperation(). The correct fix was to avoid calling AccessControlHandler.canDiscloseInformation() where it did not make sense at all, when there was no info to protect yet. LocalBackend*Operation.java: Do not call the setResultCodeAndMessageNoInfoDisclosure() when we fail to grab a lock, because this is not disclosing any sensitive information at that point. LocalBackendAddOperation.java: Do not call the setResultCodeAndMessageNoInfoDisclosure() when we fail to grab a lock, because this is not disclosing any sensitive information at that point. In lockParent(), reversed the exception to make the code easier to read. LocalBackendModifyDNOperation.java: Do not call the setResultCodeAndMessageNoInfoDisclosure() when we fail to grab a lock, because this is not disclosing any sensitive information at that point. In newDirectoryException(), removed the entryDN parameter since now this method is always called with a non null entry. AciHandler.java: Code cleanup git-svn-id: https://svn.forgerock.org/opendj/trunk@9943 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9942 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
ReplicationServerDomain.java: Fixed NPEs in stopStatusAnalyzer() and stopMonitoringPublisher(). In getEligibleCSN(), removed useless null checks. ServerWriter.java: Code cleanup. git-svn-id: https://svn.forgerock.org/opendj/trunk@9941 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9940 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9939 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
Test cleanup: stop the indexer thread after each test method has finished running. TestListener.java: Automatic code cleanup. git-svn-id: https://svn.forgerock.org/opendj/trunk@9938 41b1ffd8-f28e-4786-ab96-9950f0a78031
06 Dec, 2013
2 commits
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9937 41b1ffd8-f28e-4786-ab96-9950f0a78031
-
git-svn-id: https://svn.forgerock.org/opendj/trunk@9936 41b1ffd8-f28e-4786-ab96-9950f0a78031