Skip to content

Prepare for Test Tool v2 Release #1958

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

Merged
merged 46 commits into from
Jan 30, 2025
Merged

Prepare for Test Tool v2 Release #1958

merged 46 commits into from
Jan 30, 2025

Conversation

philasmar
Copy link
Contributor

Description of changes:
Prepare for Test Tool v2 Release

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

normj and others added 30 commits November 13, 2024 11:13
feat: use System.CommandLine to create a CLI application
feat: minor test tool v2 UI update
feat: add code editor to properly display json
…ening function and support for RequestResponse invocation type. (#1912)
Merge dev into lambda test tool branch
@philasmar philasmar added the Release Not Needed Add this label if a PR does not need to be released. label Jan 30, 2025
@philasmar philasmar requested review from normj and 96malhar January 30, 2025 20:20
@philasmar philasmar changed the title Feature/lambdatesttool v2 Prepare for Test Tool v2 Release Jan 30, 2025
{
name = name.Substring(name.IndexOf("@") + 1);
var path = Path.Combine(GetSavedRequestDirectory(), name);
return File.ReadAllText(path);

Check warning

Code scanning / Semgrep OSS

Semgrep Finding: csharp.lang.security.filesystem.unsafe-path-combine.unsafe-path-combine Warning

String argument name is used to read or write data from a file via Path.Combine without direct sanitization via Path.GetFileName. If the path is user-supplied data this can lead to path traversal.
@philasmar philasmar closed this Jan 30, 2025
@philasmar philasmar reopened this Jan 30, 2025
@philasmar philasmar merged commit f8920f9 into dev Jan 30, 2025
5 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release Not Needed Add this label if a PR does not need to be released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants