Skip to content

Commit

Permalink
Merge pull request #98 from baoduy/develop
Browse files Browse the repository at this point in the history
Develop
  • Loading branch information
baoduy authored Oct 11, 2024
2 parents 4ac8b85 + b33cf86 commit e92382c
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 11 deletions.
4 changes: 2 additions & 2 deletions src/Aks/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -266,8 +266,8 @@ export default async ({
},

azurePolicy: { enabled: true },
kubeDashboard: { enabled: false },
httpApplicationRouting: { enabled: false },
// kubeDashboard: { enabled: false },
// httpApplicationRouting: { enabled: false },

aciConnectorLinux: {
enabled: Boolean(network.virtualHostSubnetName),
Expand Down
27 changes: 18 additions & 9 deletions src/VNet/FirewallPolicies/AksFirewallPolicy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -127,15 +127,24 @@ export default ({
],
destinationPorts: ['443'],
},
// {
// ruleType: 'NetworkRule',
// name: 'others-dns',
// description: 'Others DNS.',
// ipProtocols: ['TCP', 'UDP'],
// sourceAddresses: subnetSpaces,
// destinationAddresses: ['*'],
// destinationPorts: ['53'],
// },
{
ruleType: 'NetworkRule',
name: 'aks-allows-commons-dns',
description: 'Others DNS.',
ipProtocols: ['TCP', 'UDP'],
sourceAddresses: ['*'],
destinationAddresses: [
//Azure
'168.63.129.16',
//CloudFlare
'1.1.1.1',
'1.0.0.1',
//Google
'8.8.8.8',
'8.8.4.4',
],
destinationPorts: ['53'],
},
);

//AKS Apps Rules
Expand Down

0 comments on commit e92382c

Please sign in to comment.