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 deployment process to support non-root users #1156

Open
wants to merge 1 commit into
base: canary
Choose a base branch
from

Conversation

vishwamartur
Copy link

Related to #1126

Add error handling for deployment failures with non-root users.

  • apps/api/src/utils.ts

    • Throw an error with a message when deployment fails.
  • apps/dokploy/server/utils/deploy.ts

    • Throw an error with a message when deployment fails.
  • apps/dokploy/server/api/routers/server.ts

    • Add a check for sudo password prompts in the serverSetup function and throw an error if detected.
    • Emit an error in the observable if a sudo password prompt is detected during server setup.

Related to Dokploy#1126

Add error handling for deployment failures with non-root users.

* **apps/api/src/utils.ts**
  - Throw an error with a message when deployment fails.

* **apps/dokploy/server/utils/deploy.ts**
  - Throw an error with a message when deployment fails.

* **apps/dokploy/server/api/routers/server.ts**
  - Add a check for sudo password prompts in the `serverSetup` function and throw an error if detected.
  - Emit an error in the observable if a sudo password prompt is detected during server setup.
@Siumauricio
Copy link
Contributor

This is throwing errors like this and not in the logs, can you add a video testing this?

Screenshot 2025-01-26 at 2 14 53 PM

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