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

Guard against users using relative pathing to set the project name to the .dme in the repository #1835

Merged
merged 8 commits into from
Jul 26, 2024

Conversation

Cyberboss
Copy link
Member

@Cyberboss Cyberboss commented Jul 25, 2024

PRs I shouldn't have to make.

🆑
You can no longer deploy a .dme file outside of the deployment directory.
Fixed Windows path normalization applying to Linux instances in the same swarm.
/🆑

🆑 HTTP API
Added ErrorCode 107 for if a deployment is attempted with a project outside the deployment directory.
/🆑

@Cyberboss Cyberboss added Client Issue with the .NET client library Fix Fixes incorrect functionality REST The JSON REST API for server control Area: Compiler With regard to managing the deployment process labels Jul 25, 2024
@Cyberboss Cyberboss added this to the v6.8.0 milestone Jul 25, 2024
github-actions[bot]
github-actions bot previously approved these changes Jul 25, 2024
github-actions[bot]
github-actions bot previously approved these changes Jul 25, 2024
@Cyberboss Cyberboss added the Security Issue pertaining to Authentication/Authorization or NTC (Never Trust the Client) label Jul 25, 2024
@Cyberboss Cyberboss enabled auto-merge July 25, 2024 01:00
@Cyberboss Cyberboss disabled auto-merge July 25, 2024 01:29
@Cyberboss Cyberboss enabled auto-merge July 25, 2024 01:29
github-actions[bot]
github-actions bot previously approved these changes Jul 25, 2024
github-actions[bot]
github-actions bot previously approved these changes Jul 26, 2024
github-actions[bot]
github-actions bot previously approved these changes Jul 26, 2024
@Cyberboss Cyberboss added the Swarm Related to server swarm feature label Jul 26, 2024
@Cyberboss Cyberboss disabled auto-merge July 26, 2024 15:50
@Cyberboss Cyberboss enabled auto-merge July 26, 2024 15:50
Copy link

codecov bot commented Jul 26, 2024

Codecov Report

Attention: Patch coverage is 98.73418% with 1 line in your changes missing coverage. Please review.

Project coverage is 95.95%. Comparing base (2cc3b1c) to head (3acd25e).
Report is 25 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1835      +/-   ##
==========================================
- Coverage   95.97%   95.95%   -0.02%     
==========================================
  Files         713      713              
  Lines      163357   163366       +9     
  Branches     3229     3226       -3     
==========================================
- Hits       156784   156762      -22     
- Misses       6085     6114      +29     
- Partials      488      490       +2     

@Cyberboss Cyberboss merged commit 374852f into dev Jul 26, 2024
101 checks passed
@Cyberboss Cyberboss deleted the BarGoesUpInFlames branch July 26, 2024 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Compiler With regard to managing the deployment process Client Issue with the .NET client library Fix Fixes incorrect functionality REST The JSON REST API for server control Security Issue pertaining to Authentication/Authorization or NTC (Never Trust the Client) Swarm Related to server swarm feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant