starred/temps

Fork 0

mirror of https://github.com/gotempsh/temps.git synced 2026-04-24 22:06:00 +03:00

[GH-ISSUE #26] Dockerfile path not saved #4

New issue

Closed

opened 2026-03-02 05:12:26 +03:00 by kerem · 8 comments

kerem commented

2026-03-02 05:12:26 +03:00

Owner

Originally created by @bjoerge on GitHub (Feb 28, 2026).
Original GitHub issue: https://github.com/gotempsh/temps/issues/26

Description

Changing Dockerfile path under project settings doesn't seem to have any effect:

Looking in the network tab, it doesn't seem like the API call to save settings include the changed dockerFilePath field value at all:

fetch("/api/projects/6/settings", {
  "headers": {
    "content-type": "application/json",
  },
  "body": "{\"slug\":\"my-project\"}",
  "method": "POST",
  //…
});

Steps to reproduce

Chose Docker preset on a project
Go to project settings
Change Dockerfile path to a custom dockerfile path
Reload page

Expected behavior

Expected custon dockerfile path to take effect

Actual behavior

Custom docker file path wasn't saved

Operating System

Linux

Temps version

0.0.5

Docker version

29.2.1

Relevant logs

Additional context

No response

Originally created by @bjoerge on GitHub (Feb 28, 2026). Original GitHub issue: https://github.com/gotempsh/temps/issues/26 ### Description Changing Dockerfile path under project settings doesn't seem to have any effect: <img width="685" height="446" alt="Image" src="https://github.com/user-attachments/assets/d6a19b05-5ee4-476b-89b6-ccd9eb7ec7f5" /> Looking in the network tab, it doesn't seem like the API call to save settings include the changed dockerFilePath field value at all: ```javascript fetch("/api/projects/6/settings", { "headers": { "content-type": "application/json", }, "body": "{\"slug\":\"my-project\"}", "method": "POST", //… }); ``` ### Steps to reproduce - Chose Docker preset on a project - Go to project settings - Change Dockerfile path to a custom dockerfile path - Reload page ### Expected behavior Expected custon dockerfile path to take effect ### Actual behavior Custom docker file path wasn't saved ### Operating System Linux ### Temps version 0.0.5 ### Docker version 29.2.1 ### Relevant logs ```shell ``` ### Additional context _No response_

kerem

2026-03-02 05:12:26 +03:00

closed this issue
added the
bug
label

kerem commented

2026-03-02 05:12:27 +03:00

Author

Owner

@dviejokfs commented on GitHub (Feb 28, 2026):

I'm looking into it. I think the dockerfile configuration must be near preset, which right now is in the GIT section inside the project detail, not the best UX, but the presets available are linked with the git repository.

Dockerfile path must be configurable only when preset "Dockerfile" is selected

@dviejokfs commented on GitHub (Feb 28, 2026): I'm looking into it. I think the dockerfile configuration must be near preset, which right now is in the GIT section inside the project detail, not the best UX, but the presets available are linked with the git repository. Dockerfile path must be configurable only when preset "Dockerfile" is selected

kerem commented

2026-03-02 05:12:27 +03:00

Author

Owner

@dviejokfs commented on GitHub (Feb 28, 2026):

@bjoerge the PR has been merged into main :)

@dviejokfs commented on GitHub (Feb 28, 2026): @bjoerge the PR has been merged into main :)

kerem commented

2026-03-02 05:12:27 +03:00

Author

Owner

@bjoerge commented on GitHub (Mar 1, 2026):

Thanks so much! Any plans on making a release (or is there a way to upgrade to main)?

@bjoerge commented on GitHub (Mar 1, 2026): Thanks so much! Any plans on making a release (or is there a way to upgrade to `main`)?

kerem commented

2026-03-02 05:12:28 +03:00

Author

Owner

@dviejokfs commented on GitHub (Mar 1, 2026):

I’ll release today v0.0.6-beta.1 @bjoerge

@dviejokfs commented on GitHub (Mar 1, 2026): I’ll release today v0.0.6-beta.1 @bjoerge

kerem commented

2026-03-02 05:12:28 +03:00

Author

Owner

@dviejokfs commented on GitHub (Mar 1, 2026):

V0.0.6-beta.1 tag pushed @bjoerge

@dviejokfs commented on GitHub (Mar 1, 2026): V0.0.6-beta.1 tag pushed @bjoerge

kerem commented

2026-03-02 05:12:28 +03:00

Author

Owner

@bjoerge commented on GitHub (Mar 1, 2026):

Thanks, looks like the tag is missing some build assets:
curl -fsSL https://temps.sh/upgrade.sh | bash -s v0.0.6-beta.1

results in

…
  • Downloading new Temps binary...

curl: (22) The requested URL returned error: 404

error: Failed to download temps from "https://github.com/gotempsh/temps/releases/download/v0.0.6-beta.1/temps-linux-amd64.tar.gz"

@bjoerge commented on GitHub (Mar 1, 2026): Thanks, looks like the tag is missing some build assets: ```curl -fsSL https://temps.sh/upgrade.sh | bash -s v0.0.6-beta.1``` results in ``` … • Downloading new Temps binary... curl: (22) The requested URL returned error: 404 error: Failed to download temps from "https://github.com/gotempsh/temps/releases/download/v0.0.6-beta.1/temps-linux-amd64.tar.gz" ```

kerem commented

2026-03-02 05:12:28 +03:00

Author

Owner

@dviejokfs commented on GitHub (Mar 1, 2026):

yeah this pipeline needs to finish in order to be available: https://github.com/gotempsh/temps/actions/runs/22542353973 @bjoerge

@dviejokfs commented on GitHub (Mar 1, 2026): yeah this pipeline needs to finish in order to be available: https://github.com/gotempsh/temps/actions/runs/22542353973 @bjoerge

kerem commented

2026-03-02 05:12:28 +03:00

Author

Owner

@bjoerge commented on GitHub (Mar 1, 2026):

Worked now, thanks!

@bjoerge commented on GitHub (Mar 1, 2026): Worked now, thanks!

kerem referenced this issue

2026-03-02 05:12:30 +03:00

[PR #4] [MERGED] fix(proxy): remove internal ID headers from proxy responses #6