Skip to content

Comments

DOSTM-8643_CDN_migration#55

Merged
anshuman-mor merged 3 commits intomasterfrom
DOSTM-8643_CDN_migration
Jan 30, 2026
Merged

DOSTM-8643_CDN_migration#55
anshuman-mor merged 3 commits intomasterfrom
DOSTM-8643_CDN_migration

Conversation

@anshuman-mor
Copy link
Collaborator

This pull request updates how tool filenames are handled, especially for tools whose CDN and local filenames differ. It introduces helper functions to consistently resolve the correct filenames, updates the download logic to use these helpers, and changes the tool download URL and API prefix to match new requirements. Additionally, the dependency on @digicert/ssm-client-tools-installer is bumped to version 1.6.0.

Filename handling improvements:

  • Introduced getLocalFileName and getCdnFileName helper functions to resolve tool filenames, supporting tools with different CDN and local filenames. Updated all relevant logic to use these helpers instead of directly accessing toolDownloaded. (build/index.js) [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14]

Download logic and API changes:

  • Changed the API prefix and host for tool downloads to new values, and updated the download URL construction to use the CDN filename with proper encoding. (build/index.js) [1] [2]
  • Updated the download and extraction logic to work with the new filename resolution and URL patterns. (build/index.js) [1] [2]

Dependency updates:

  • Bumped @digicert/ssm-client-tools-installer dependency from version 1.5.0 to 1.6.0. (package.json)

Updated from branch reference to published version containing CDN migration changes.

Related to DOSTM-8643
Copy link
Collaborator

@UtkarshBagariaDigicert UtkarshBagariaDigicert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@anshuman-mor anshuman-mor merged commit af7f4b0 into master Jan 30, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants