fixing adding new policy not shown error

merge-requests/1/head
Rasika Perera 9 years ago
parent dadf913a83
commit 6ea6551ada

@ -32,7 +32,7 @@ if (uriMatcher.match("/{context}/api/policies/update")) {
policyModule.updatePolicyPriorities(payload);
} else if (uriMatcher.match("/{context}/api/policies/add")) {
var content = request.getContent();
var id = content.policyName;
var policyName = content.policyName;
var policyDefinition = content.profile.policyDefinition;
var policyDescription = content.profile.policyDescription;
var deviceType = content.profile.deviceType.name;
@ -40,7 +40,7 @@ if (uriMatcher.match("/{context}/api/policies/update")) {
log.info("@@@@@ Policy Declaration : " + stringify(content));
try {
result = policyModule.addPolicy(id, deviceType, policyDefinition, policyDescription);
result = policyModule.addPolicy(policyName, deviceType, policyDefinition, policyDescription);
} catch (e) {
log.error("Exception occurred while trying to add new policy under name:" + id, e);
// http status code 500 refers to - Internal Server Error.

@ -27,9 +27,7 @@ policyModule = function () {
var privateMethods = {};
publicMethods.addPolicy = function (policyName, deviceType, policyDefinition, policyDescription) {
if(policyName && deviceType){
return false;
}
if (policyName && deviceType) {
var carbonModule = require("carbon");
var carbonServer = application.get("carbonServer");
@ -59,7 +57,12 @@ policyModule = function () {
var result = mqttsender.pushToMQTT("/iot/policymgt/govern", policyDefinition, "tcp://10.100.0.104:1883", "Raspberry-Policy-sender");
mqttsender = null;
return true;
} else {
return false;
}
};
publicMethods.getPolicies = function () {
@ -82,7 +85,7 @@ policyModule = function () {
log.info("Policies for device types: " + allPolicies.content);
log.info("");
var deviceType = allPolicies.content[i].replace(constants.POLICY_REGISTRY_PATH, "");
log.info("##### deviceType:"+deviceType);
log.info("##### deviceType:" + deviceType);
var deviceTypePolicies = registry.get(allPolicies.content[i]);
//loop through policies
@ -96,7 +99,7 @@ policyModule = function () {
//"profile": {}, // Profile
"policyName": deviceTypePolicy.name, // Name of the policy.
"updated": deviceTypePolicy.updated.time,
"deviceType":deviceType
"deviceType": deviceType
//"generic": true, // If true, this should be applied to all related device.
//"roles": {}, // Roles which this policy should be applied.
//"ownershipType": {}, // Ownership type (COPE, BYOD, CPE)
@ -139,7 +142,7 @@ policyModule = function () {
try {
registry.remove(constants.POLICY_REGISTRY_PATH + deviceType + "/" + name);
bool = true;
}catch(err){
} catch (err) {
log.error("Error while trying to remove policy :" + name);
}
}

Loading…
Cancel
Save