Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

Commit

Permalink
run the container as the current user, not root
Browse files Browse the repository at this point in the history
  • Loading branch information
dmathieu committed Dec 8, 2017
1 parent 7d2d878 commit 74cf021
Show file tree
Hide file tree
Showing 4 changed files with 932 additions and 3 deletions.
1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ $ heroku plugins:install heroku-container-registry
In a directory with a Dockerfile:

```
$ heroku container:run web
$ heroku container:login
$ heroku create
$ heroku container:push
Expand Down
2 changes: 1 addition & 1 deletion commands/run.js
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ let run = async function (context, heroku) {
jobs = possibleJobs.standard || []
}
if (!jobs.length) {
cli.warn('No images to push')
cli.warn('No images to run')
process.exit(1)
}

Expand Down
3 changes: 2 additions & 1 deletion lib/sanbashi.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
let Glob = require('glob')
let Path = require('path')
let Inquirer = require('inquirer')
let os = require("os")
const Child = require('child_process')
const log = require('./log')

Expand Down Expand Up @@ -102,7 +103,7 @@ Sanbashi.pushImage = function (resource, verbose) {
}

Sanbashi.runImage = function (resource, command, verbose) {
let args = ['run']
let args = ['run', '--user', os.userInfo().uid]
if (command == '') {
args.push(resource)
} else {
Expand Down
Loading

0 comments on commit 74cf021

Please sign in to comment.