Skip to content

Commit

Permalink
removing notions of a log model
Browse files Browse the repository at this point in the history
  • Loading branch information
evantahler committed Feb 7, 2012
1 parent b78bfb7 commit 4ae08d4
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 15 deletions.
12 changes: 0 additions & 12 deletions initializers/initActions.js
Expand Up @@ -23,18 +23,6 @@ var initActions = function(api, next)
next();
});

api.logAction = function(api, connection){
if(api.models != null && api.models.log != null){
var logRecord = api.models.log.build({
ip: connection.remoteIP,
action: connection.action,
error: connection.error,
params: JSON.stringify(connection.params)
});
process.nextTick(function() { logRecord.save(); });
}
}

api.processAction = function(api, connection, next){
var templateValidator = require('validator').Validator;
connection.validator = new templateValidator();
Expand Down
1 change: 0 additions & 1 deletion initializers/initSocketServer.js
Expand Up @@ -209,7 +209,6 @@ var initSocketServer = function(api, next){
api.socketServer.sendSocketMessage(connection, connection.response);
}
}
process.nextTick(function() { api.logAction(api, connection); });
}

////////////////////////////////////////////////////////////////////////////
Expand Down
1 change: 0 additions & 1 deletion initializers/initWebServer.js
Expand Up @@ -137,7 +137,6 @@ var initWebServer = function(api, next)
connection.res.end(stringResponse);
}
if(api.configData.logRequests){api.log(" > web request from " + connection.remoteIP + " | responded in : " + connection.response.serverInformation.requestDuration + "ms", "grey");}
process.nextTick(function() { api.logAction(api, connection); });
});
};

Expand Down
4 changes: 3 additions & 1 deletion versions.markdown
Expand Up @@ -2,10 +2,12 @@

## Version 0.2.4

**Summary:** Goodbye Rate Limiting
**Summary:** Goodbye Rate Limiting, validators, and XML

**Details:**

* Actions will be checked for the proper variables and methods.
* Actions' required params will be automatically checked for existence.
* Removed anything to do with rate-limiting. I don't think that anyone was using it anyway. This really isn't a core feature and was causing trouble at the DB layer.
* api will no longer return details of requests remaining
* DBs no longer need to build the `api.rateLimitCheck` method
Expand Down

0 comments on commit 4ae08d4

Please sign in to comment.