inteligr8/alfresco-community-repo
1
0
Fork 0
mirror of https://github.com/Alfresco/alfresco-community-repo.git synced 2025-08-07 17:49:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

ALF-7871 - org.alfresco.error.AlfrescoRuntimeException: 022286935 Read-Write transaction started within read-only transaction

Browse Source 
git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@27133 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
Mark Rogers
2011-04-28 12:35:58 +00:00
parent 411dccf795
commit 4653b40026
1 changed files with 25 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
source/java/org/alfresco/filesys/auth/cifs/CifsAuthenticatorBase.java
View File

@@ -39,6 +39,7 @@ import org.alfresco.repo.security.authentication.AuthenticationUtil;
import org.alfresco.repo.security.authentication.MD4PasswordEncoder; import org.alfresco.repo.security.authentication.MD4PasswordEncoder;
import org.alfresco.repo.security.authentication.MD4PasswordEncoderImpl; import org.alfresco.repo.security.authentication.MD4PasswordEncoderImpl;
import org.alfresco.repo.security.authentication.ntlm.NLTMAuthenticator; import org.alfresco.repo.security.authentication.ntlm.NLTMAuthenticator;
import org.alfresco.repo.transaction.AlfrescoTransactionSupport;
import org.alfresco.repo.transaction.RetryingTransactionHelper; import org.alfresco.repo.transaction.RetryingTransactionHelper;
import org.alfresco.service.cmr.repository.NodeRef; import org.alfresco.service.cmr.repository.NodeRef;
import org.alfresco.service.cmr.repository.NodeService; import org.alfresco.service.cmr.repository.NodeService;
@@ -407,24 +408,26 @@ public abstract class CifsAuthenticatorBase extends CifsAuthenticator implements
/** /**
* Set the current authenticated user context for this thread. * Set the current authenticated user context for this thread.
* *
* @param client * @param client ClientInfo or null to clear the context
* ClientInfo
*/ */
public void setCurrentUser(final ClientInfo client) { public void setCurrentUser(final ClientInfo client) {
// Check the account type and setup the authentication context // Check the account type and setup the authentication context
doInTransaction(new RetryingTransactionHelper.RetryingTransactionCallback<Object>() // No need for a transaction to clear the context
{
public Object execute() throws Throwable
{
if (client == null || client.isNullSession()) if (client == null || client.isNullSession())
{ {
// Clear the authentication, null user should not be allowed to do any service calls // Clear the authentication, null user should not be allowed to do any service calls
getAuthenticationComponent().clearCurrentSecurityContext(); getAuthenticationComponent().clearCurrentSecurityContext();
return;
} }
else if (client.isGuest() == false && client instanceof AlfrescoClientInfo)
doInTransaction(new RetryingTransactionHelper.RetryingTransactionCallback<Object>()
{
public Object execute() throws Throwable
{
if (client.isGuest() == false && client instanceof AlfrescoClientInfo)
{ {
// Set the authentication context for the request // Set the authentication context for the request
@@ -577,9 +580,21 @@ public abstract class CifsAuthenticatorBase extends CifsAuthenticator implements
// DEBUG // DEBUG
if (logger.isDebugEnabled()) if (logger.isDebugEnabled())
{
logger.debug("Using " + (txService.isReadOnly() ? "ReadOnly" : "Write") + " transaction"); logger.debug("Using " + (txService.isReadOnly() ? "ReadOnly" : "Write") + " transaction");
}
//
// the repository is read-only, we settle for a read-only transaction
if (txService.isReadOnly())
{
return txService.getRetryingTransactionHelper().doInTransaction(callback, true, false);
}
// otherwise we want force a writable transaction
return txService.getRetryingTransactionHelper().doInTransaction(callback,
false,
false);
return txService.getRetryingTransactionHelper().doInTransaction(callback, txService.isReadOnly());
} }
/** /**