fix: mount correct socket on unix

[Fractal-Tess]

What kind of change does this PR introduce?

This should be mounting the correct docker socket on UNIX by using utils.Docker.DaemonHost() instead of client.DefaultDockerHost.

Fixes a bug where on rootless docker, it mounts the incorrect path of /var/run/docker.sock instead of the correct one /run/user/1000/docker.sock or whatever DOCKER_HOST

What is the current behaviour?

It tries to mount /var/run/docker.sock

Please link any relevant issues here.
#3127

What is the new behaviour?

Should mount the correct path.

Additional context

It would be awesome if someone else also tests this functionality as this could introduce bugs on other systems.

PS - this pr was largely thanks to the help of @PhatDave with testing and debugging.

[sweatybridge]

Could you double check if the logs explorer works fine in studio with rootless socket?

I vaguely remember analytics couldn't access the socket to read logs even though the container starts fine.

[coveralls]

Pull Request Test Coverage Report for Build 13300919621

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 1 of 3 (33.33%) changed or added relevant lines in 1 file are covered.
• 71 unchanged lines in 3 files lost coverage.
• Overall coverage increased (+0.004%) to 58.566%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
internal/start/start.go	1	3	33.33%

Files with Coverage Reduction	New Missed Lines	%
internal/storage/rm/rm.go	2	89.53%
internal/gen/keys/keys.go	5	12.9%
internal/start/start.go	64	65.44%

Totals
Change from base Build 13280059914:	0.004%
Covered Lines:	7811
Relevant Lines:	13337

---

💛 - Coveralls

[Fractal-Tess]

I can confirm that there are proper logs showing in the dashboard page for all of the services except for cron and edge functions.
These two show

{
  "code": 502,
  "errors": [],
  "message": "Something went wrong! Unknown error. If this continues please contact support.",
  "status": "UNKNOWN"
}

[Fractal-Tess]

Amazing, thank you very much. When do you reckon this change will be shipped to npm?

[sweatybridge]

Once PR is merged, it will be available in beta release channel.

npx supabase@beta start

Stable channel will be updated in 2 weeks.