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

fix: permission issue for default drogon uploads folder #1870

Merged
merged 1 commit into from
Feb 3, 2025

Conversation

vansangpfiev
Copy link
Contributor

Describe Your Changes

This pull request includes a change to the engine/main.cc file to enhance the server's file upload functionality. The most important change is the addition of a new upload path for file uploads.

Enhancements to server functionality:

  • engine/main.cc: Added code to set the upload path to the system's temporary directory for file uploads.

Fixes Issues

  • Closes #
  • Closes #

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

@vansangpfiev vansangpfiev merged commit 596eb94 into dev Feb 3, 2025
6 checks passed
@vansangpfiev vansangpfiev deleted the fix/perm-uploads branch February 3, 2025 02:33
vansangpfiev added a commit that referenced this pull request Feb 11, 2025
vansangpfiev added a commit that referenced this pull request Feb 11, 2025
chore: cherry-pick: permission issue for default drogon uploads folder (#1870)
vansangpfiev added a commit that referenced this pull request Feb 18, 2025
* feat: AMD hardware API (#1797)

* feat: add amd gpu windows

* chore: remove unused code

* feat: get amd gpus

* fix: clean

* chore: cleanup

* fix: set activate

* fix: build windows

* feat: linux

* fix: add patches

* fix: map cuda gpus

* fix: build

* chore: docs

* fix: build

* chore: clean up

* fix: build

* fix: build

* chore: pack vulkan windows

* chore: vulkan linux

---------

Co-authored-by: vansangpfiev <[email protected]>

* fix: add cpu usage (#1868)

Co-authored-by: vansangpfiev <[email protected]>

* fix: PATCH method for Thread and Messages management (#1923)

Co-authored-by: vansangpfiev <[email protected]>

* fix: ignore compute_cap if not present (#1866)

* fix: ignore compute_cap if not present

* fix: correct gpu info

* fix: remove check for toolkit version

---------

Co-authored-by: vansangpfiev <[email protected]>

* fix: models.cc: symlinked model deletion shouldn't remove original file (#1918)

Co-authored-by: vansangpfiev <[email protected]>

* fix: correct gpu info list (#1944)

* fix: correct gpu info list

* chore: cleanup

---------

Co-authored-by: vansangpfiev <[email protected]>

* fix: gpu: filter out llvmpipe

* fix: add vendor in gpu info (#1952)

Co-authored-by: vansangpfiev <[email protected]>

* fix: correct get server name method (#1953)

Co-authored-by: vansangpfiev <[email protected]>

* fix: map nvidia and vulkan uuid (#1954)

Co-authored-by: vansangpfiev <[email protected]>

* fix: permission issue for default drogon uploads folder (#1870)

Co-authored-by: vansangpfiev <[email protected]>

* chore: change timeout

* fix: make get hardware info function thread-safe (#1956)

Co-authored-by: vansangpfiev <[email protected]>

* fix: cache data for gpu information (#1959)

* fix: wrap vulkan gpu function

* fix: init

* fix: cpu usage

* fix: build windows

* fix: buld macos

---------

Co-authored-by: vansangpfiev <[email protected]>

* fix: handle path with space (#1963)

* fix: unload engine before updating (#1970)

Co-authored-by: sangjanai <[email protected]>

* fix: auto-reload model for remote engine (#1971)

Co-authored-by: sangjanai <[email protected]>

* fix: use updated configuration for remote model when reload (#1972)

Co-authored-by: sangjanai <[email protected]>

* fix: correct engine interface order (#1974)

Co-authored-by: sangjanai <[email protected]>

* fix: improve error handling for remote engine (#1975)

Co-authored-by: sangjanai <[email protected]>

* fix: temporarily remove model setting recommendation (#1977)

Co-authored-by: sangjanai <[email protected]>

---------

Co-authored-by: vansangpfiev <[email protected]>
Co-authored-by: OHaiiBuzzle <[email protected]>
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.

3 participants