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.

Sign up for GitHub

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

Jump to bottom

Composer helper : allow run as other user #2367

Open
Salamandar opened this issue Mar 24, 2024 · 2 comments · May be fixed by YunoHost/yunohost#1808
Open

Composer helper : allow run as other user #2367

Salamandar opened this issue Mar 24, 2024 · 2 comments · May be fixed by YunoHost/yunohost#1808
Labels
🤓 helpers 📦 App packaging 👾 bug Something isn't working

Comments

@Salamandar
Copy link

Salamandar commented Mar 24, 2024 •
edited

Describe the bug

# ynh_composer_exec
[...]
Do not run Composer as root/super user! See https://getcomposer.org/root for details
Aborting as no plugin should be loaded if running as super user is not explicitly allowed

# ynh_exec_as "$app" ynh_composer_exec
sudo: ynh_composer_exec: command not found
@Salamandar Salamandar added the 👾 bug Something isn't working label Mar 24, 2024
@tituspijean
Copy link

I think I had at some point coded an alternative composer helper for Flarum with that fearure. We could reuse that.

@OniriCorpe
Copy link
Member

maybe run it as $app by default or mandatory?

@OniriCorpe OniriCorpe linked a pull request Mar 25, 2024 that will close this issue
Open
@OniriCorpe OniriCorpe linked a pull request Mar 25, 2024 that will close this issue
helpers: run composer as the app user YunoHost/yunohost#1808
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🤓 helpers 📦 App packaging 👾 bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

helpers: run composer as the app user YunoHost/yunohost
3 participants
@Salamandar @OniriCorpe @tituspijean