Skip to content

Commit

Permalink
Merge pull request #35 from alertlogic/dereg_aws
Browse files Browse the repository at this point in the history
Pass AWS collector properties for deregistration
  • Loading branch information
kkuzmin authored Dec 5, 2019
2 parents f25e277 + ad4fe05 commit 3b3f18f
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 3 deletions.
2 changes: 1 addition & 1 deletion azcollectc.js
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ class AzcollectC extends AlServiceC {
const type = this._collectorType;
var functionName = encodeURIComponent(registrationValues.functionName);
return this.deleteRequest(`/aws/${type}/` +
`${registrationValues.awsAccountId}/${registrationValues.region}/${functionName}`);
`${registrationValues.awsAccountId}/${registrationValues.region}/${functionName}`, {body: registrationValues});
}

_doCheckinAzure(checkinInput) {
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@alertlogic/al-collector-js",
"version": "1.4.1",
"version": "1.4.2",
"license": "MIT",
"description": "Alert Logic Collector Common Library",
"repository": {
Expand Down
5 changes: 4 additions & 1 deletion test/azcollectc_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,10 @@ describe('Unit Tests', function() {
};

azc.deregister(checkinValues).then( resp => {
sinon.assert.calledWith(fakeDel, '/aws/cwe/1234567890/us-east-1/test-function');
sinon.assert.calledWith(fakeDel, '/aws/cwe/1234567890/us-east-1/test-function',
{
body: checkinValues
});
done();
});
});
Expand Down

0 comments on commit 3b3f18f

Please sign in to comment.