Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
suggested fix

Co-authored-by: Fatima <66124862+fatima99s@users.noreply.github.com>
  • Loading branch information
AkhtarAmir and fatima99s authored Nov 7, 2024
1 parent 7f6b3ce commit f4c3d30
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions plugins/azure/sqldatabases/dbTDEEnabled.js
Original file line number Diff line number Diff line change
Expand Up @@ -51,9 +51,11 @@ module.exports = {
databases.data.forEach(database => {

if (database.name && database.name.toLowerCase() !== 'master') {
var transparentDataEncryption = helpers.addSource(cache, source, ['transparentDataEncryption', 'list', location, database.id]);
var transparentDataEncryption = helpers.addSource(cache, source,
['transparentDataEncryption', 'list', location, database.id]);

Check failure on line 55 in plugins/azure/sqldatabases/dbTDEEnabled.js

View workflow job for this annotation

GitHub Actions / build

Expected indentation of 44 spaces but found 40

if (!transparentDataEncryption || transparentDataEncryption.err || !transparentDataEncryption.data || !transparentDataEncryption.data.length) {
if (!transparentDataEncryption || transparentDataEncryption.err ||
!transparentDataEncryption.data || !transparentDataEncryption.data.length) {
helpers.addResult(results, 3, 'Unable to query transparent data encryption for SQL Database: ' + helpers.addError(transparentDataEncryption), location, database.id);
return;
}
Expand Down

0 comments on commit f4c3d30

Please sign in to comment.