More refactoring and a workaround to support the iPhone/android way of authenticating.
This commit is contained in:
parent
ec9691ecd3
commit
8832cc2374
@ -1,5 +1,8 @@
|
|||||||
package se.su.dsv.scipro.json;
|
package se.su.dsv.scipro.json;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Interface for delegated callback handling of response-strings.
|
||||||
|
*/
|
||||||
public interface IResponseHandler {
|
public interface IResponseHandler {
|
||||||
public void handleResponse(String response);
|
public void handleResponse(String response);
|
||||||
}
|
}
|
||||||
|
@ -3,6 +3,7 @@ package se.su.dsv.scipro.json;
|
|||||||
import java.lang.reflect.Type;
|
import java.lang.reflect.Type;
|
||||||
|
|
||||||
import org.apache.log4j.Level;
|
import org.apache.log4j.Level;
|
||||||
|
import org.apache.log4j.Logger;
|
||||||
import org.springframework.stereotype.Component;
|
import org.springframework.stereotype.Component;
|
||||||
|
|
||||||
import se.su.dsv.scipro.jsonobjects.JsonUpdateStatsContainer;
|
import se.su.dsv.scipro.jsonobjects.JsonUpdateStatsContainer;
|
||||||
@ -17,6 +18,7 @@ import com.google.gson.reflect.TypeToken;
|
|||||||
*/
|
*/
|
||||||
@Component
|
@Component
|
||||||
public class ImportUpdateStatsResponseHandler extends JsonResponseHandler {
|
public class ImportUpdateStatsResponseHandler extends JsonResponseHandler {
|
||||||
|
private Logger logger = Logger.getLogger(ImportUpdateStatsResponseHandler.class);
|
||||||
/**
|
/**
|
||||||
* Handles a json string
|
* Handles a json string
|
||||||
*
|
*
|
||||||
|
@ -3,6 +3,7 @@ package se.su.dsv.scipro.json;
|
|||||||
import java.lang.reflect.Type;
|
import java.lang.reflect.Type;
|
||||||
|
|
||||||
import org.apache.log4j.Level;
|
import org.apache.log4j.Level;
|
||||||
|
import org.apache.log4j.Logger;
|
||||||
import org.springframework.stereotype.Component;
|
import org.springframework.stereotype.Component;
|
||||||
|
|
||||||
import se.su.dsv.scipro.data.dataobjects.Project;
|
import se.su.dsv.scipro.data.dataobjects.Project;
|
||||||
@ -24,6 +25,7 @@ import com.google.gson.reflect.TypeToken;
|
|||||||
*/
|
*/
|
||||||
@Component
|
@Component
|
||||||
public class ImportWorkerResponseHandler extends JsonResponseHandler {
|
public class ImportWorkerResponseHandler extends JsonResponseHandler {
|
||||||
|
private Logger logger = Logger.getLogger(ImportWorkerResponseHandler.class);
|
||||||
/**
|
/**
|
||||||
* Handles a json string
|
* Handles a json string
|
||||||
*
|
*
|
||||||
@ -123,7 +125,7 @@ public class ImportWorkerResponseHandler extends JsonResponseHandler {
|
|||||||
}
|
}
|
||||||
|
|
||||||
if(isLogResult()){
|
if(isLogResult()){
|
||||||
logger.log(Level.INFO, "\nResult from userimport/update:\nChecked: " + completeContainer.users.size() + " users\n" +
|
logger.info("\nResult from userimport/update:\nChecked: " + completeContainer.users.size() + " users\n" +
|
||||||
"Changed users: " + changedUsers + "\n" +
|
"Changed users: " + changedUsers + "\n" +
|
||||||
"Created users: " + createdUsers + "\n" +
|
"Created users: " + createdUsers + "\n" +
|
||||||
"Number of errors when creating users: " + userCreationErrors + "\n" +
|
"Number of errors when creating users: " + userCreationErrors + "\n" +
|
||||||
|
@ -52,7 +52,7 @@ abstract class JsonResponseHandler implements IResponseHandler {
|
|||||||
@Autowired
|
@Autowired
|
||||||
protected ApplicationSettings applicationSettings;
|
protected ApplicationSettings applicationSettings;
|
||||||
|
|
||||||
protected Logger logger = Logger.getLogger(JsonResponseHandler.class);
|
private Logger logger = Logger.getLogger(JsonResponseHandler.class);
|
||||||
|
|
||||||
private int numUsersCreated = 0;
|
private int numUsersCreated = 0;
|
||||||
private int numEmailChanges = 0;
|
private int numEmailChanges = 0;
|
||||||
|
@ -3,6 +3,7 @@ package se.su.dsv.scipro.json;
|
|||||||
import java.lang.reflect.Type;
|
import java.lang.reflect.Type;
|
||||||
|
|
||||||
import org.apache.log4j.Level;
|
import org.apache.log4j.Level;
|
||||||
|
import org.apache.log4j.Logger;
|
||||||
import org.springframework.stereotype.Component;
|
import org.springframework.stereotype.Component;
|
||||||
|
|
||||||
import se.su.dsv.scipro.data.dataobjects.User;
|
import se.su.dsv.scipro.data.dataobjects.User;
|
||||||
@ -22,6 +23,7 @@ import com.google.gson.reflect.TypeToken;
|
|||||||
*/
|
*/
|
||||||
@Component
|
@Component
|
||||||
public class JsonUserFullResponseHandler extends JsonResponseHandler {
|
public class JsonUserFullResponseHandler extends JsonResponseHandler {
|
||||||
|
private Logger logger = Logger.getLogger(JsonUserFullResponseHandler.class);
|
||||||
/**
|
/**
|
||||||
* Handles a json string
|
* Handles a json string
|
||||||
*
|
*
|
||||||
|
@ -2,6 +2,7 @@ package se.su.dsv.scipro.json;
|
|||||||
|
|
||||||
import java.lang.reflect.Type;
|
import java.lang.reflect.Type;
|
||||||
|
|
||||||
|
import org.apache.log4j.Logger;
|
||||||
import org.springframework.stereotype.Component;
|
import org.springframework.stereotype.Component;
|
||||||
|
|
||||||
import se.su.dsv.scipro.data.dataobjects.User;
|
import se.su.dsv.scipro.data.dataobjects.User;
|
||||||
@ -19,6 +20,7 @@ import com.google.gson.reflect.TypeToken;
|
|||||||
*/
|
*/
|
||||||
@Component
|
@Component
|
||||||
public class JsonUserResponseHandler extends JsonResponseHandler {
|
public class JsonUserResponseHandler extends JsonResponseHandler {
|
||||||
|
private Logger logger = Logger.getLogger(JsonUserResponseHandler.class);
|
||||||
/**
|
/**
|
||||||
* Handles a json string
|
* Handles a json string
|
||||||
*
|
*
|
||||||
|
@ -34,7 +34,8 @@ public class JsonKeyAuthPage extends WebPage {
|
|||||||
|
|
||||||
if (jsonLogin != null) {
|
if (jsonLogin != null) {
|
||||||
String loginUsername = jsonLogin.username;
|
String loginUsername = jsonLogin.username;
|
||||||
User user = userDao.getUserByUsername(loginUsername);
|
//@ToDo Fix the apple-app to support non hard-coded realm
|
||||||
|
User user = userDao.getUserByUsername(loginUsername,"DSV.SU.SE");
|
||||||
|
|
||||||
if (user == null) {
|
if (user == null) {
|
||||||
jsonOutput.addProperty("authenticated", false);
|
jsonOutput.addProperty("authenticated", false);
|
||||||
|
@ -69,9 +69,9 @@ public class JsonLoginPage extends WebPage {
|
|||||||
}
|
}
|
||||||
|
|
||||||
if (loginUsername != null) {
|
if (loginUsername != null) {
|
||||||
|
|
||||||
String username = jsonLogin.username;
|
String username = jsonLogin.username;
|
||||||
User user = userDao.getUserByUsername(username);
|
//@ToDo Fix the apple-app to support non hard-coded realm
|
||||||
|
User user = userDao.getUserByUsername(username,"DSV.SU.SE");
|
||||||
if (roleDao.isEmployee(user)) {
|
if (roleDao.isEmployee(user)) {
|
||||||
jsonOutput.addProperty("authenticated", true);
|
jsonOutput.addProperty("authenticated", true);
|
||||||
UserSettings userSettings = userSettingsDao.getUserSettings(user);
|
UserSettings userSettings = userSettingsDao.getUserSettings(user);
|
||||||
|
Loading…
x
Reference in New Issue
Block a user