Skip to content
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.

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

Quote launch process command paths #239

Merged
merged 8 commits into from
Mar 19, 2025

Conversation

runesoerensen
Copy link
Contributor

This PR primarily ensures that paths used in launch processes containing special characters are properly escaped and shell compatible.

@runesoerensen runesoerensen force-pushed the quote-launch-process-command-paths branch from 0301a90 to 1ae41a3 Compare March 19, 2025 09:11
@runesoerensen runesoerensen changed the base branch from main to sanitize-launch-process-type March 19, 2025 09:11
Base automatically changed from sanitize-launch-process-type to main March 19, 2025 09:25
@runesoerensen runesoerensen force-pushed the quote-launch-process-command-paths branch from 1ae41a3 to 967ac95 Compare March 19, 2025 09:27
@runesoerensen runesoerensen marked this pull request as ready for review March 19, 2025 09:28
@runesoerensen runesoerensen requested a review from a team as a code owner March 19, 2025 09:28
@runesoerensen runesoerensen merged commit 1807f21 into main Mar 19, 2025
6 checks passed
@runesoerensen runesoerensen deleted the quote-launch-process-command-paths branch March 19, 2025 10:10
heroku-linguist bot added a commit that referenced this pull request Mar 20, 2025
## heroku/dotnet

### Changed

- The buildpack now sanitizes launch process type names, based on project assembly names, by filtering out invalid characters. ([#237](#237))
- Launch process commands with paths containing special characters (including spaces) are now properly quoted. ([#239](#239))
- The `test` launch process, added when targeting the test execution environment, now properly handles solution/project filenames containing special characters (including spaces). ([#240](#240))
@heroku-linguist heroku-linguist bot mentioned this pull request Mar 20, 2025
heroku-linguist bot added a commit that referenced this pull request Mar 20, 2025
## heroku/dotnet

### Changed

- The buildpack now sanitizes launch process type names, based on project assembly names, by filtering out invalid characters. ([#237](#237))
- Launch process commands with paths containing special characters (including spaces) are now properly quoted. ([#239](#239))
- The `test` launch process, added when targeting the test execution environment, now properly handles solution/project filenames containing special characters (including spaces). ([#240](#240))

Co-authored-by: heroku-linguist[bot] <136119646+heroku-linguist[bot]@users.noreply.github.com>
heroku-linguist bot added a commit to heroku/cnb-builder-images that referenced this pull request Mar 20, 2025
## heroku/dotnet

### Changed

- The buildpack now sanitizes launch process type names, based on project assembly names, by filtering out invalid characters. ([#237](heroku/buildpacks-dotnet#237))
- Launch process commands with paths containing special characters (including spaces) are now properly quoted. ([#239](heroku/buildpacks-dotnet#239))
- The `test` launch process, added when targeting the test execution environment, now properly handles solution/project filenames containing special characters (including spaces). ([#240](heroku/buildpacks-dotnet#240))
heroku-linguist bot added a commit to heroku/cnb-builder-images that referenced this pull request Mar 20, 2025
## heroku/dotnet

### Changed

- The buildpack now sanitizes launch process type names, based on project assembly names, by filtering out invalid characters. ([#237](heroku/buildpacks-dotnet#237))
- Launch process commands with paths containing special characters (including spaces) are now properly quoted. ([#239](heroku/buildpacks-dotnet#239))
- The `test` launch process, added when targeting the test execution environment, now properly handles solution/project filenames containing special characters (including spaces). ([#240](heroku/buildpacks-dotnet#240))

Co-authored-by: heroku-linguist[bot] <136119646+heroku-linguist[bot]@users.noreply.github.com>
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