-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add a create/update/changeId/delete posthook (#193) #242
Changes from 4 commits
6d51dfe
2f2ecb6
68cd544
d9271d4
45d8a32
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
@@ -0,0 +1,171 @@ | ||||||||||||||||||||||||||
/* | ||||||||||||||||||||||||||
**************************************************************************** | ||||||||||||||||||||||||||
* Ldap Synchronization Connector provides tools to synchronize | ||||||||||||||||||||||||||
* electronic identities from a list of data sources including | ||||||||||||||||||||||||||
* any database with a JDBC connector, another LDAP directory, | ||||||||||||||||||||||||||
* flat files... | ||||||||||||||||||||||||||
* | ||||||||||||||||||||||||||
* ==LICENSE NOTICE== | ||||||||||||||||||||||||||
* | ||||||||||||||||||||||||||
* Copyright (c) 2008 - 2011 LSC Project | ||||||||||||||||||||||||||
* All rights reserved. | ||||||||||||||||||||||||||
* | ||||||||||||||||||||||||||
* Redistribution and use in source and binary forms, with or without | ||||||||||||||||||||||||||
* modification, are permitted provided that the following conditions are met: | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
* * Redistributions of source code must retain the above copyright | ||||||||||||||||||||||||||
* notice, this list of conditions and the following disclaimer. | ||||||||||||||||||||||||||
* * Redistributions in binary form must reproduce the above copyright | ||||||||||||||||||||||||||
* notice, this list of conditions and the following disclaimer in the | ||||||||||||||||||||||||||
* documentation and/or other materials provided with the distribution. | ||||||||||||||||||||||||||
* * Neither the name of the LSC Project nor the names of its | ||||||||||||||||||||||||||
* contributors may be used to endorse or promote products derived from | ||||||||||||||||||||||||||
* this software without specific prior written permission. | ||||||||||||||||||||||||||
* | ||||||||||||||||||||||||||
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS | ||||||||||||||||||||||||||
* IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED | ||||||||||||||||||||||||||
* TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A | ||||||||||||||||||||||||||
* PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER | ||||||||||||||||||||||||||
* OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, | ||||||||||||||||||||||||||
* EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, | ||||||||||||||||||||||||||
* PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR | ||||||||||||||||||||||||||
* PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF | ||||||||||||||||||||||||||
* LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING | ||||||||||||||||||||||||||
* NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS | ||||||||||||||||||||||||||
* SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | ||||||||||||||||||||||||||
* | ||||||||||||||||||||||||||
* ==LICENSE NOTICE== | ||||||||||||||||||||||||||
* | ||||||||||||||||||||||||||
* (c) 2008 - 2011 LSC Project | ||||||||||||||||||||||||||
**************************************************************************** | ||||||||||||||||||||||||||
*/ | ||||||||||||||||||||||||||
package org.lsc; | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
import org.slf4j.Logger; | ||||||||||||||||||||||||||
import org.slf4j.LoggerFactory; | ||||||||||||||||||||||||||
import java.lang.ProcessBuilder; | ||||||||||||||||||||||||||
import java.io.OutputStream; | ||||||||||||||||||||||||||
import java.io.IOException; | ||||||||||||||||||||||||||
import java.util.Optional; | ||||||||||||||||||||||||||
import org.lsc.utils.output.LdifLayout; | ||||||||||||||||||||||||||
import org.lsc.beans.syncoptions.ISyncOptions.OutputFormat; | ||||||||||||||||||||||||||
import com.fasterxml.jackson.databind.ObjectMapper; // For encoding object to JSON | ||||||||||||||||||||||||||
import com.fasterxml.jackson.databind.ObjectWriter; | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
/** | ||||||||||||||||||||||||||
* This object is managing posthook scripts | ||||||||||||||||||||||||||
*/ | ||||||||||||||||||||||||||
public class Hooks { | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
static final Logger LOGGER = LoggerFactory.getLogger(Hooks.class); | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
private static final ObjectMapper Mapper = new ObjectMapper(); | ||||||||||||||||||||||||||
/** | ||||||||||||||||||||||||||
* Method calling a postSyncHook if necessary | ||||||||||||||||||||||||||
* | ||||||||||||||||||||||||||
* return nothing | ||||||||||||||||||||||||||
*/ | ||||||||||||||||||||||||||
public final void postSyncHook(final Optional<String> hook, final OutputFormat outputFormat, final LscModifications lm) { | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
if( hook != null && hook.isPresent() ) | ||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||
switch (lm.getOperation()) { | ||||||||||||||||||||||||||
case CREATE_OBJECT: | ||||||||||||||||||||||||||
callHook("create", hook.get(), lm.getMainIdentifier(), outputFormat, lm); | ||||||||||||||||||||||||||
break; | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
case UPDATE_OBJECT: | ||||||||||||||||||||||||||
callHook("update", hook.get(), lm.getMainIdentifier(), outputFormat, lm); | ||||||||||||||||||||||||||
break; | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
case CHANGE_ID: | ||||||||||||||||||||||||||
callHook("changeId", hook.get(), lm.getMainIdentifier(), outputFormat, lm); | ||||||||||||||||||||||||||
break; | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
case DELETE_OBJECT: | ||||||||||||||||||||||||||
callHook("delete", hook.get(), lm.getMainIdentifier(), outputFormat, lm); | ||||||||||||||||||||||||||
break; | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
default: | ||||||||||||||||||||||||||
LOGGER.info("Error: unknown operation for posthook {}", hook); | ||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
public final static void callHook( String operationType, | ||||||||||||||||||||||||||
String hook, | ||||||||||||||||||||||||||
String identifier, | ||||||||||||||||||||||||||
OutputFormat outputFormat, | ||||||||||||||||||||||||||
LscModifications lm) { | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
LOGGER.info("Calling {} posthook {} with format {} for {}", | ||||||||||||||||||||||||||
operationType, | ||||||||||||||||||||||||||
hook, | ||||||||||||||||||||||||||
outputFormat.toString(), | ||||||||||||||||||||||||||
identifier); | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
String modifications = null; | ||||||||||||||||||||||||||
// Compute modifications only in a create / update / changeid operation | ||||||||||||||||||||||||||
if( ! operationType.equals("delete") ) | ||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||
if( outputFormat == OutputFormat.JSON ) { | ||||||||||||||||||||||||||
modifications = getJsonModifications(lm); | ||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||
else { | ||||||||||||||||||||||||||
modifications = LdifLayout.format(lm); | ||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
try { | ||||||||||||||||||||||||||
if( modifications != null ) { | ||||||||||||||||||||||||||
Process p = new ProcessBuilder( | ||||||||||||||||||||||||||
hook, | ||||||||||||||||||||||||||
identifier, | ||||||||||||||||||||||||||
operationType) | ||||||||||||||||||||||||||
.start(); | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
// sends ldif modifications to stdin of hook script | ||||||||||||||||||||||||||
OutputStream stdin = p.getOutputStream(); | ||||||||||||||||||||||||||
stdin.write(modifications.getBytes()); | ||||||||||||||||||||||||||
stdin.write("\n".getBytes()); | ||||||||||||||||||||||||||
stdin.flush(); | ||||||||||||||||||||||||||
stdin.close(); | ||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||
else { | ||||||||||||||||||||||||||
Process p = new ProcessBuilder( | ||||||||||||||||||||||||||
hook, | ||||||||||||||||||||||||||
identifier, | ||||||||||||||||||||||||||
operationType) | ||||||||||||||||||||||||||
.start(); | ||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||
catch(IOException e) { | ||||||||||||||||||||||||||
LOGGER.error("Error while calling {} posthook {} with format {} for {}", | ||||||||||||||||||||||||||
operationType, | ||||||||||||||||||||||||||
hook, | ||||||||||||||||||||||||||
outputFormat.toString(), | ||||||||||||||||||||||||||
identifier); | ||||||||||||||||||||||||||
LOGGER.error("posthook error: ", e); | ||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
should work also |
||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
/** | ||||||||||||||||||||||||||
* Method computing modifications as json | ||||||||||||||||||||||||||
* | ||||||||||||||||||||||||||
* @return modifications in a json String | ||||||||||||||||||||||||||
*/ | ||||||||||||||||||||||||||
public final static String getJsonModifications(final LscModifications lm) { | ||||||||||||||||||||||||||
ObjectWriter ow = Mapper.writer().withDefaultPrettyPrinter(); | ||||||||||||||||||||||||||
String json = ""; | ||||||||||||||||||||||||||
try { | ||||||||||||||||||||||||||
json = ow.writeValueAsString(lm.getLscAttributeModifications()); | ||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||
catch(Exception e) { | ||||||||||||||||||||||||||
LOGGER.error("Error while encoding LSC modifications to json"); | ||||||||||||||||||||||||||
LOGGER.error("encoding error: ", e); | ||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||
return json; | ||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
} |
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -47,6 +47,7 @@ | |||||
|
||||||
import java.util.List; | ||||||
import java.util.Set; | ||||||
import java.util.Optional; | ||||||
|
||||||
import org.lsc.LscModificationType; | ||||||
import org.lsc.configuration.PolicyType; | ||||||
|
@@ -128,6 +129,30 @@ public String getCondition(LscModificationType operation) { | |||||
return DEFAULT_CONDITION; | ||||||
} | ||||||
|
||||||
public OutputFormat getPostHookOutputFormat() { | ||||||
return OutputFormat.LDIF; | ||||||
} | ||||||
|
||||||
public Optional<String> getCreatePostHook() { | ||||||
return Optional.ofNullable(null); | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
} | ||||||
|
||||||
public Optional<String> getDeletePostHook() { | ||||||
return Optional.ofNullable(null); | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
} | ||||||
|
||||||
public Optional<String> getUpdatePostHook() { | ||||||
return Optional.ofNullable(null); | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
} | ||||||
|
||||||
public Optional<String> getChangeIdPostHook() { | ||||||
return Optional.ofNullable(null); | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
} | ||||||
|
||||||
public Optional<String> getPostHook(LscModificationType operation) { | ||||||
return Optional.ofNullable(null); | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
} | ||||||
|
||||||
public String getDn() { | ||||||
return null;//((Ldap)task.getDestinationService()).getDn(); | ||||||
} | ||||||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -49,6 +49,7 @@ | |
import java.util.HashSet; | ||
import java.util.List; | ||
import java.util.Set; | ||
import java.util.Optional; | ||
|
||
import org.lsc.LscModificationType; | ||
import org.lsc.configuration.DatasetType; | ||
|
@@ -196,6 +197,58 @@ public String getCondition(LscModificationType operation) { | |
} | ||
return result; | ||
} | ||
|
||
public OutputFormat getPostHookOutputFormat() { | ||
// default: returns LDIF format | ||
if (conf.getHooks() == null || conf.getHooks().getOutputFormat() == null) { | ||
return OutputFormat.LDIF; | ||
} | ||
switch(conf.getHooks().getOutputFormat()){ | ||
case "json": | ||
return OutputFormat.JSON; | ||
default: | ||
return OutputFormat.LDIF; | ||
} | ||
} | ||
|
||
public Optional<String> getCreatePostHook() { | ||
Optional<String> hook = Optional.ofNullable(conf.getHooks().getCreatePostHook()).filter(s -> !s.isEmpty()); | ||
return hook; | ||
} | ||
|
||
public Optional<String> getDeletePostHook() { | ||
Optional<String> hook = Optional.ofNullable(conf.getHooks().getDeletePostHook()).filter(s -> !s.isEmpty()); | ||
return hook; | ||
} | ||
|
||
public Optional<String> getUpdatePostHook() { | ||
Optional<String> hook = Optional.ofNullable(conf.getHooks().getUpdatePostHook()).filter(s -> !s.isEmpty()); | ||
return hook; | ||
} | ||
|
||
public Optional<String> getChangeIdPostHook() { | ||
Optional<String> hook = Optional.ofNullable(conf.getHooks().getChangeIdPostHook()).filter(s -> !s.isEmpty()); | ||
return hook; | ||
} | ||
|
||
public Optional<String> getPostHook(LscModificationType operation) { | ||
Optional<String> result = Optional.ofNullable(null); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. you can simplify this method by removing the result method and returning directly into each case |
||
switch (operation) { | ||
case CREATE_OBJECT: | ||
result = this.getCreatePostHook(); | ||
break; | ||
case UPDATE_OBJECT: | ||
result = this.getUpdatePostHook(); | ||
break; | ||
case DELETE_OBJECT: | ||
result = this.getDeletePostHook(); | ||
break; | ||
case CHANGE_ID: | ||
result = this.getChangeIdPostHook(); | ||
break; | ||
} | ||
return result; | ||
} | ||
|
||
public String getDelimiter(String name) { | ||
DatasetType dataset = LscConfiguration.getDataset(conf, name); | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,6 +6,7 @@ | |
import java.util.List; | ||
import java.util.Map; | ||
import java.util.Map.Entry; | ||
import java.util.Optional; | ||
|
||
import javax.naming.CommunicationException; | ||
|
||
|
@@ -23,6 +24,8 @@ | |
import org.lsc.utils.ScriptingEvaluator; | ||
import org.slf4j.Logger; | ||
import org.slf4j.LoggerFactory; | ||
import org.lsc.Hooks; | ||
import org.lsc.beans.syncoptions.ISyncOptions.OutputFormat; | ||
|
||
/** | ||
* @author sfroger | ||
|
@@ -108,6 +111,11 @@ public void run() { | |
// if we got here, we have a modification to apply - let's | ||
// do it! | ||
if (task.getDestinationService().apply(lm)) { | ||
// Retrieve posthook for the current operation | ||
Optional<String> hook = syncOptions.getDeletePostHook(); | ||
OutputFormat outputFormat = syncOptions.getPostHookOutputFormat(); | ||
Hooks hookObject = new Hooks(); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. hookObject (you can name it hooks) should be initialized in the constructor. |
||
hookObject.postSyncHook(hook, outputFormat, lm); | ||
counter.incrementCountCompleted(); | ||
abstractSynchronize.logAction(lm, id, task.getName()); | ||
} else { | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks better.
The principle of an optional is to not be null, so you can remove the null check.
Also if instead of a String operationType you continue using LscModificationType, the code can be very simplified by: