X-Git-Url: http://git.nikiroo.be/?a=blobdiff_plain;f=src%2Fbe%2Fnikiroo%2Ffanfix%2Flibrary%2FRemoteLibraryServer.java;h=d62b41b0283e5b2fe49abe2a47059e98ce8e6bca;hb=70fa286e0932aca313a5596b0eeee8218e86c24d;hp=c0b05064c2ec4a7915ee2b26393766ac377c0047;hpb=9f51d8ab092b891f407dc1bfe0e0fa1cef7ff944;p=nikiroo-utils.git
diff --git a/src/be/nikiroo/fanfix/library/RemoteLibraryServer.java b/src/be/nikiroo/fanfix/library/RemoteLibraryServer.java
index c0b0506..d62b41b 100644
--- a/src/be/nikiroo/fanfix/library/RemoteLibraryServer.java
+++ b/src/be/nikiroo/fanfix/library/RemoteLibraryServer.java
@@ -19,40 +19,47 @@ import be.nikiroo.utils.serial.server.ConnectActionServerObject;
import be.nikiroo.utils.serial.server.ServerObject;
/**
- * Create a new remote server that will listen for order on the given port.
+ * Create a new remote server that will listen for orders on the given port.
*
- * The available commands are given as arrays of objects (first item is the key,
- * second is the command, the rest are the arguments).
+ * The available commands are given as arrays of objects (first item is the
+ * command, the rest are the arguments).
*
- * The md5 is always a String (the MD5 hash of the access key), the commands are
- * also Strings; the parameters vary depending upon the command.
+ * All commands, including PING, will first return a random value to you that
+ * you must hash with your key and return before processing the rest; if the
+ * value is OK, it will return "true", if not, it will return NULL and stop the
+ * connection.
+ *
+ * BTW: this system is by no means secure. It is just slightly
+ * obfuscated, and operate on clear text (because Google decided not to support
+ * anonymous SSL exchanges on Android, and the main use case for this server is
+ * Android).
*
- * - [md5] PING: will return PONG if the key is accepted
- * - [md5] GET_METADATA *: will return the metadata of all the stories in the
- * library (array)
- * *
- * - [md5] GET_METADATA [luid]: will return the metadata of the story of LUID
+ *
- PING: will return PONG if the key is accepted
+ * - GET_METADATA *: will return the metadata of all the stories in the
+ * library (array)
*
+ * - GET_METADATA [luid]: will return the metadata of the story of LUID
* luid
- * - [md5] GET_STORY [luid]: will return the given story if it exists (or NULL
- * if not)
- * - [md5] SAVE_STORY [luid]: save the story (that must be sent just after the
+ *
- GET_STORY [luid]: will return the given story if it exists (or NULL if
+ * not)
+ * - SAVE_STORY [luid]: save the story (that must be sent just after the
* command) with the given LUID, then return the LUID
- * - [md5] IMPORT [url]: save the story found at the given URL, then return
- * the LUID
- * - [md5] DELETE_STORY [luid]: delete the story of LUID luid
- * - [md5] GET_COVER [luid]: return the cover of the story
- * - [md5] GET_CUSTOM_SOURCE_COVER [source]: return the cover for this source
- * - [md5] SET_SOURCE_COVER [source], [luid]: set the default cover for the
- * given source to the cover of the story denoted by luid
- * - [md5] CHANGE_SOURCE [luid] [new source]: change the source of the story
- * of LUID luid
- * - [md5] EXIT: stop the server
+ * - IMPORT [url]: save the story found at the given URL, then return the
+ * LUID
+ * - DELETE_STORY [luid]: delete the story of LUID luid
+ * - GET_COVER [luid]: return the cover of the story
+ * - GET_CUSTOM_COVER ["SOURCE"|"AUTHOR"] [source]: return the cover for this
+ * source/author
+ * - SET_COVER ["SOURCE"|"AUTHOR"] [value] [luid]: set the default cover for
+ * the given source/author to the cover of the story denoted by luid
+ * - CHANGE_SOURCE [luid] [new source]: change the source of the story of LUID
+ * luid
+ * - EXIT: stop the server
*
*
* @author niki
*/
public class RemoteLibraryServer extends ServerObject {
- private final String md5;
+ private final String key;
/**
* Create a new remote server (will not be active until
@@ -67,8 +74,8 @@ public class RemoteLibraryServer extends ServerObject {
* in case of I/O error
*/
public RemoteLibraryServer(String key, int port) throws IOException {
- super("Fanfix remote library", port, true);
- this.md5 = StringUtils.getMd5Hash(key);
+ super("Fanfix remote library", port, false);
+ this.key = key;
setTraceHandler(Instance.getTraceHandler());
}
@@ -76,39 +83,49 @@ public class RemoteLibraryServer extends ServerObject {
@Override
protected Object onRequest(ConnectActionServerObject action,
Version clientVersion, Object data) throws Exception {
- String md5 = "";
+ long start = new Date().getTime();
+
String command = "";
Object[] args = new Object[0];
if (data instanceof Object[]) {
Object[] dataArray = (Object[]) data;
if (dataArray.length >= 2) {
- md5 = "" + dataArray[0];
- command = "" + dataArray[1];
+ command = "" + dataArray[0];
- args = new Object[dataArray.length - 2];
- for (int i = 2; i < dataArray.length; i++) {
- args[i - 2] = dataArray[i];
+ args = new Object[dataArray.length - 1];
+ for (int i = 1; i < dataArray.length; i++) {
+ args[i - 1] = dataArray[i];
}
}
}
- String trace = "[" + command + "] ";
+ String trace = "[ " + command + "] ";
for (Object arg : args) {
trace += arg + " ";
}
getTraceHandler().trace(trace);
- if (!md5.equals(this.md5)) {
+ // Authentication:
+ String random = StringUtils.getMd5Hash(Double.toString(Math.random()));
+ action.send(random);
+ String answer = "";
+ try {
+ answer += action.rec();
+ } catch (NullPointerException e) {
+ return null;
+ }
+
+ if (answer.equals(RemoteLibrary.hashKey(key, random))) {
+ action.send(true);
+ } else {
getTraceHandler().trace("Key rejected.");
return null;
}
- long start = new Date().getTime();
Object rep = doRequest(action, command, args);
- getTraceHandler().trace(
- String.format("[/%s]: %d ms", command,
- (new Date().getTime() - start)));
+ getTraceHandler().trace(String.format("[>%s]: %d ms", command,
+ (new Date().getTime() - start)));
return rep;
}
@@ -123,6 +140,7 @@ public class RemoteLibraryServer extends ServerObject {
Progress pg = createPgForwarder(action);
List metas = new ArrayList();
+
for (MetaData meta : Instance.getLibrary().getMetas(pg)) {
MetaData light;
if (meta.getCover() == null) {
@@ -139,8 +157,8 @@ public class RemoteLibraryServer extends ServerObject {
return metas.toArray(new MetaData[] {});
}
- return new MetaData[] { Instance.getLibrary().getInfo(
- (String) args[0]) };
+ return new MetaData[] {
+ Instance.getLibrary().getInfo((String) args[0]) };
} else if ("GET_STORY".equals(command)) {
MetaData meta = Instance.getLibrary().getInfo((String) args[0]);
meta = meta.clone();
@@ -149,8 +167,8 @@ public class RemoteLibraryServer extends ServerObject {
action.send(meta);
action.rec();
- Story story = Instance.getLibrary()
- .getStory((String) args[0], null);
+ Story story = Instance.getLibrary().getStory((String) args[0],
+ null);
for (Object obj : breakStory(story)) {
action.send(obj);
action.rec();
@@ -171,23 +189,36 @@ public class RemoteLibraryServer extends ServerObject {
return story.getMeta().getLuid();
} else if ("IMPORT".equals(command)) {
Progress pg = createPgForwarder(action);
- Story story = Instance.getLibrary().imprt(
- new URL((String) args[0]), pg);
+ Story story = Instance.getLibrary().imprt(new URL((String) args[0]),
+ pg);
forcePgDoneSent(pg);
return story.getMeta().getLuid();
} else if ("DELETE_STORY".equals(command)) {
Instance.getLibrary().delete((String) args[0]);
} else if ("GET_COVER".equals(command)) {
return Instance.getLibrary().getCover((String) args[0]);
- } else if ("GET_CUSTOM_SOURCE_COVER".equals(command)) {
- return Instance.getLibrary().getCustomSourceCover((String) args[0]);
- } else if ("SET_SOURCE_COVER".equals(command)) {
- Instance.getLibrary().setSourceCover((String) args[0],
- (String) args[1]);
- } else if ("CHANGE_SOURCE".equals(command)) {
+ } else if ("GET_CUSTOM_COVER".equals(command)) {
+ if ("SOURCE".equals(args[0])) {
+ return Instance.getLibrary()
+ .getCustomSourceCover((String) args[1]);
+ } else if ("AUTHOR".equals(args[0])) {
+ return Instance.getLibrary()
+ .getCustomAuthorCover((String) args[1]);
+ } else {
+ return null;
+ }
+ } else if ("SET_COVER".equals(command)) {
+ if ("SOURCE".equals(args[0])) {
+ Instance.getLibrary().setSourceCover((String) args[1],
+ (String) args[2]);
+ } else if ("AUTHOR".equals(args[0])) {
+ Instance.getLibrary().setAuthorCover((String) args[1],
+ (String) args[2]);
+ }
+ } else if ("CHANGE_STA".equals(command)) {
Progress pg = createPgForwarder(action);
- Instance.getLibrary().changeSource((String) args[0],
- (String) args[1], pg);
+ Instance.getLibrary().changeSTA((String) args[0], (String) args[1],
+ (String) args[2], (String) args[3], pg);
forcePgDoneSent(pg);
} else if ("EXIT".equals(command)) {
stop(0, false);
@@ -310,9 +341,8 @@ public class RemoteLibraryServer extends ServerObject {
public void progress(Progress progress, String name) {
int min = pg.getMin();
int max = pg.getMax();
- int relativeProgress = min
- + (int) Math.round(pg.getRelativeProgress()
- * (max - min));
+ int relativeProgress = min + (int) Math
+ .round(pg.getRelativeProgress() * (max - min));
// Do not re-send the same value twice over the wire,
// unless more than 2 seconds have elapsed (to maintain the
@@ -324,15 +354,17 @@ public class RemoteLibraryServer extends ServerObject {
p[2] = relativeProgress;
try {
- action.send(new Integer[] { min, max, relativeProgress });
+ action.send(
+ new Integer[] { min, max, relativeProgress });
action.rec();
} catch (Exception e) {
Instance.getTraceHandler().error(e);
}
- isDoneForwarded[0] = pg.isDone();
lastTime[0] = new Date().getTime();
}
+
+ isDoneForwarded[0] = (pg.getProgress() >= pg.getMax());
}
});