Skip to content

fix: gohttpserver checksum verification fails due to wrong working directory#26

Open
Copilot wants to merge 3 commits intomainfrom
copilot/add-gohttpserver-file-management
Open

fix: gohttpserver checksum verification fails due to wrong working directory#26
Copilot wants to merge 3 commits intomainfrom
copilot/add-gohttpserver-file-management

Conversation

Copy link

Copilot AI commented Mar 20, 2026

sha256sum --check resolves filenames relative to CWD (/app), but the tarball is downloaded to /tmp/, causing a "No such file or directory" error and build failure.

Changes

  • Dockerfile: Replace sha256sum --check pipe with explicit hash comparison — fetch the expected digest from the checksums file, compute the actual digest against the full /tmp/ path, and fail loudly on mismatch:
EXPECTED=$(curl -fsSL ".../gohttpserver_1.3.0_checksums.txt" \
  | grep "${TARBALL}" | awk '{print $1}') && \
ACTUAL=$(sha256sum "/tmp/${TARBALL}" | awk '{print $1}') && \
[ "$EXPECTED" = "$ACTUAL" ] || (echo "Checksum mismatch for ${TARBALL}" && exit 1)
Original prompt

This section details on the original issue you should resolve

<issue_title>Add gohttpserver for web-based file management</issue_title>
<issue_description>In order to manage OpenClaw effectively, access to the gateway is given, and access to ttyd (browser-based terminal access) also.

To complete ease of management we need to give users access in the same way to a file manager so they can change the files themselves. Gohttpserver seems to be an accepted solution.

</issue_description>

<agent_instructions>Add gohttpserver and expose it and then give a link in /dappnode_package.json like in here: "links": {
"homepage": "https://openclaw.ai",
"ui": "http://openclaw.dappnode:18789#token=openclaw",
"terminal": "http://openclaw.dappnode:7681",
"setup-wizard": "http://openclaw.dappnode:8080",
"docs": "https://docs.openclaw.ai"

So users can easily access the gohttpserver by clicking this link.

Implementation should be very similar to ttyd</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: Pol-Lanski <19168735+Pol-Lanski@users.noreply.github.com>
Copilot AI changed the title [WIP] Add gohttpserver for web-based file management Add gohttpserver for web-based file management Mar 20, 2026
Copilot AI requested a review from Pol-Lanski March 20, 2026 04:12
@Pol-Lanski Pol-Lanski marked this pull request as ready for review March 20, 2026 04:19
Co-authored-by: Pol-Lanski <19168735+Pol-Lanski@users.noreply.github.com>
Copilot AI changed the title Add gohttpserver for web-based file management fix: gohttpserver checksum verification fails due to wrong working directory Mar 20, 2026
@github-actions
Copy link
Contributor

Dappnode bot has built and pinned the built packages to an IPFS node, for commit: 2ffd997

This is a development version and should only be installed for testing purposes.

  1. Package openclaw.dnp.dappnode.eth

Install link

Hash: /ipfs/QmcD3fs5NrUEx5Pt5GeY4eYhSMUkBmjoFkxWtpoodQrjB1

(by dappnodebot/build-action)

@Pol-Lanski Pol-Lanski requested a review from hcastc00 March 20, 2026 06:48
Copy link
Member

@Pol-Lanski Pol-Lanski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested the new package on my Dappnode and works!
Not tested in ARM

Image

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.

Add gohttpserver for web-based file management

2 participants