Skip to content

fix: check hostname for profile endpoint url #37

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
anisaoshafi merged 5 commits into from
Sep 8, 2025
Merged

fix: check hostname for profile endpoint url #37

Changes from 4 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3e8fd70
Style: override -> Override
anisaoshafi Sep 6, 2025
1cd146c
Handle https protocol for endpoint_url
anisaoshafi Sep 6, 2025
8ceae9b
Refactor aws profile code
anisaoshafi Sep 6, 2025
6677c0a
Update src/utils/configure-aws.ts
anisaoshafi Sep 8, 2025
f0444b1
Update src/utils/configure-aws.ts
anisaoshafi Sep 8, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
50 changes: 32 additions & 18 deletions src/utils/configure-aws.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,15 +14,15 @@
// TODO: add a test for this.

const LOCALSTACK_CONFIG_PROFILE_NAME = "profile localstack";
const VALID_ENDPOINT_URLS = [
"http://localhost.localstack.cloud:4566", // default
"http://127.0.0.1:4566",
"http://localhost:4566",
const VALID_HOSTNAMES = [
"localhost.localstack.cloud",
"127.0.0.1",
"localhost",
];
const DEFAULT_PORT = "4566";
const LOCALSTACK_CONFIG_PROPERTIES = {
region: "us-east-1",
output: "json",
endpoint_url: VALID_ENDPOINT_URLS,
};

// https://docs.aws.amazon.com/cli/v1/userguide/cli-configure-files.html
Expand All @@ -40,16 +40,30 @@
override: boolean = false,
): Promise<OverrideDecision | undefined> {
if (override === true) {
return "override";
return "Override";
}
const fileList = filesToModify.join(" and ");
const selection = await window.showWarningMessage(
`The "localstack" AWS profile in ${fileList} exists, but does not match the expected properties. Do you want to override it?`,
"override",
"Override",
);
return selection;
}

function isValidEndpointUrl(url: string | undefined): boolean {
if (!url) return false;
try {
const parsed = new URL(url);
return (

Check failure on line 57 in src/utils/configure-aws.ts

View workflow job for this annotation

GitHub Actions / Test 

Type 'string | false' is not assignable to type 'boolean'.
(parsed.protocol === "http:" || parsed.protocol === "https:") &&
VALID_HOSTNAMES.includes(parsed.hostname) &&
parsed.port // port must be present
Comment thread
anisaoshafi marked this conversation as resolved.
Outdated
);
} catch {
return false;
}
}

function checkIfConfigNeedsOverride(section: IniSection | undefined): boolean {
if (!section) {
return true; // profile doesn't exist
Expand All @@ -58,7 +72,7 @@
return !(
section.properties.region &&
section.properties.endpoint_url &&
VALID_ENDPOINT_URLS.includes(section.properties.endpoint_url)
isValidEndpointUrl(section.properties.endpoint_url)
);
}

Expand Down Expand Up @@ -93,7 +107,7 @@
}
}

type OverrideDecision = "override" | "do_not_override";
type OverrideDecision = "Override" | "do_not_override";

async function configureAwsConfigProfile(
awsConfigFilename: string,
Expand All @@ -110,14 +124,14 @@
try {
if (section) {
// LocalStack profile exists, but does not match the expected properties
if (overrideDecision === "override") {
if (overrideDecision === "Override") {
// User chose to override the existing profile.

// check if dnsResolveCheck is successful
const isDnsResolved = await dnsResolveCheck();
const endpointUrl = isDnsResolved
? "http://localhost.localstack.cloud:4566"
: VALID_ENDPOINT_URLS[1];
? `http://localhost.localstack.cloud:${DEFAULT_PORT}`
: `http://127.0.0.1:${DEFAULT_PORT}`;

const updatedIniFile = updateIniSection(
iniFile,
Expand All @@ -143,8 +157,8 @@
// check if dnsResolveCheck is successful
const isDnsResolved = await dnsResolveCheck();
const endpointUrl = isDnsResolved
? "http://localhost.localstack.cloud:4566"
: VALID_ENDPOINT_URLS[1];
? `http://localhost.localstack.cloud:${DEFAULT_PORT}`
: `http://127.0.0.1:${DEFAULT_PORT}`;

const updatedIniFile = updateIniSection(
iniFile,
Expand Down Expand Up @@ -186,7 +200,7 @@
try {
// LocalStack profile exists, but does not match the expected properties
if (section) {
if (overrideDecision === "override") {
if (overrideDecision === "Override") {
// User chose to override the existing profile.
const updatedIniFile = updateIniSection(
iniFile,
Expand Down Expand Up @@ -321,7 +335,7 @@
// profiles are there but need adjustment
// in testing, we always override
if (options?.forceOverride) {
overrideDecision = "override";
overrideDecision = "Override";
} else {
// check which files need override
const filesToModify = [];
Expand All @@ -333,7 +347,7 @@
}
} else {
// if any of the profiles don't exist, we need to create it
overrideDecision = "override";
overrideDecision = "Override";
}

if (overrideDecision === undefined) {
Expand All @@ -350,7 +364,7 @@
},
});
return;
} else if (overrideDecision === "override") {
} else if (overrideDecision === "Override") {
if (configNeedsOverride && credentialsNeedsOverride) {
[configModified, credentialsModified] = await Promise.all([
configureAwsConfigProfile(
Expand Down
Loading