Skip to content

Update README.md#450

Open
yannhowe wants to merge 1 commit intoCrowdStrike:mainfrom
yannhowe:patch-1
Open

Update README.md#450
yannhowe wants to merge 1 commit intoCrowdStrike:mainfrom
yannhowe:patch-1

Conversation

@yannhowe
Copy link

@yannhowe yannhowe commented Nov 5, 2025

Fix: Add sudo -E flag to installation examples to preserve environment variables

Updated all installation examples to include sudo -E flag, which is required to preserve exported environment variables when running with elevated privileges. Without this flag, users encounter AWS SSM Parameter Store errors when the script attempts to retrieve credentials that were already exported in their shell.

Changes:

  • Added sudo -E to all install script usage examples
  • Added inline comment explaining the -E flag purpose
  • Updated troubleshooting commands to include sudo -E
  • Maintains backward compatibility with direct variable passing method

This resolves a common user error where exported credentials are not passed to the script when using sudo.

**Fix: Add sudo -E flag to installation examples to preserve environment variables**

Updated all installation examples to include `sudo -E` flag, which is required to preserve exported environment variables when running with elevated privileges. Without this flag, users encounter AWS SSM Parameter Store errors when the script attempts to retrieve credentials that were already exported in their shell.

**Changes:**
- Added `sudo -E` to all install script usage examples
- Added inline comment explaining the `-E` flag purpose
- Updated troubleshooting commands to include `sudo -E`
- Maintains backward compatibility with direct variable passing method

This resolves a common user error where exported credentials are not passed to the script when using sudo.
@github-actions
Copy link

github-actions bot commented Feb 3, 2026

This pull request has been automatically marked as stale because it has had no activity in the last 14 days.

Next steps:
• Please update the PR to address any existing feedback
• Rebase your branch to resolve any conflicts with the main branch
• The PR will be closed in 7 days if no activity occurs
• Comment on this PR to prevent automatic closure

We value your contribution and look forward to reviewing your changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant