-
-
Notifications
You must be signed in to change notification settings - Fork 132
Add Upgopher file server script #1068
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.
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
base: main
Are you sure you want to change the base?
Conversation
- Add Upgopher LXC container creation script - Add installation script for Upgopher v1.11.1 - Add JSON metadata with configuration details - Upgopher is a lightweight file upload/download server written in Go
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can fully replaced with our build in function
fetch_and_deploy_gh_release "upgopher" "wanetty/upgopher" "prebuild" "latest" "/opt/upgopher" "upgopher_.*_linux_amd64.tar.gz"
ct/upgopher.sh
Outdated
| var_ram="${var_ram:-512}" | ||
| var_disk="${var_disk:-4}" | ||
| var_os="${var_os:-debian}" | ||
| var_version="${var_version:-12}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
make it against deb 13
apt-get --> apt
use deb822 format for apt lists
ct/upgopher.sh
Outdated
| msg_ok "Stopped Services" | ||
|
|
||
| cd /opt/upgopher | ||
| RELEASE_URL=$(curl -s https://api.github.com/repos/wanetty/upgopher/releases/latest | grep "browser_download_url.*linux_amd64.tar.gz" | cut -d '"' -f 4) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use our fetch and deploy func --> https://github.com/community-scripts/ProxmoxVED/blob/main/misc/tools.func#L1372
frontend/public/json/upgopher.json
Outdated
| "ram": 512, | ||
| "hdd": 4, | ||
| "os": "Debian", | ||
| "version": "12" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
13
install/upgopher-install.sh
Outdated
| msg_info "Installing Dependencies" | ||
| $STD apt-get install -y curl wget | ||
| msg_ok "Installed Dependencies" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove, already in core deps
install/upgopher-install.sh
Outdated
| mkdir -p /opt/upgopher | ||
| cd /opt/upgopher | ||
| RELEASE_URL=$(curl -s https://api.github.com/repos/wanetty/upgopher/releases/latest | grep "browser_download_url.*linux_amd64.tar.gz" | cut -d '"' -f 4) | ||
| wget -q "$RELEASE_URL" | ||
| tar -xzf upgopher_*_linux_amd64.tar.gz | ||
| mv upgopher_*_linux_amd64/* . | ||
| rmdir upgopher_*_linux_amd64 | ||
| rm -f upgopher_*_linux_amd64.tar.gz | ||
| chmod +x upgopher |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use fetch and deploy func as above
install/upgopher-install.sh
Outdated
| # Use default configuration (no authentication, HTTP, default port/directory) | ||
| # Users can modify /etc/systemd/system/upgopher.service after installation to enable features |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove comments
install/upgopher-install.sh
Outdated
| $STD apt-get -y autoremove | ||
| $STD apt-get -y autoclean |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add apt clean -y
install/upgopher-install.sh
Outdated
| msg_info "Installing Dependencies" | ||
| $STD apt-get install -y curl wget | ||
| msg_ok "Installed Dependencies" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not needed, are core deps
ct/upgopher.sh
Outdated
| var_ram="${var_ram:-512}" | ||
| var_disk="${var_disk:-4}" | ||
| var_os="${var_os:-debian}" | ||
| var_version="${var_version:-12}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bump 13
ct/upgopher.sh
Outdated
| cd /opt/upgopher | ||
| RELEASE_URL=$(curl -s https://api.github.com/repos/wanetty/upgopher/releases/latest | grep "browser_download_url.*linux_amd64.tar.gz" | cut -d '"' -f 4) | ||
| wget -q "$RELEASE_URL" | ||
| tar -xzf upgopher_*_linux_amd64.tar.gz | ||
| mv upgopher_*_linux_amd64/* . | ||
| rmdir upgopher_*_linux_amd64 | ||
| rm -f upgopher_*_linux_amd64.tar.gz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fetch_and_deploy_gh_release "upgopher" "wanetty/upgopher" "prebuild" "latest" "/opt/upgopher" "upgopher_.*_linux_amd64.tar.gz"
ct/upgopher.sh
Outdated
| msg_info "Starting Services" | ||
| systemctl start upgopher | ||
| msg_ok "Started Services" | ||
| msg_ok "Updated Successfully" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated successfully!
frontend/public/json/upgopher.json
Outdated
| "ram": 512, | ||
| "hdd": 4, | ||
| "os": "Debian", | ||
| "version": "12" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
13
| msg_info "Installing Upgopher" | ||
| mkdir -p /opt/upgopher | ||
| cd /opt/upgopher | ||
| RELEASE_URL=$(curl -s https://api.github.com/repos/wanetty/upgopher/releases/latest | grep "browser_download_url.*linux_amd64.tar.gz" | cut -d '"' -f 4) | ||
| wget -q "$RELEASE_URL" | ||
| tar -xzf upgopher_*_linux_amd64.tar.gz | ||
| mv upgopher_*_linux_amd64/* . | ||
| rmdir upgopher_*_linux_amd64 | ||
| rm -f upgopher_*_linux_amd64.tar.gz | ||
| chmod +x upgopher | ||
| msg_ok "Installed Upgopher" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fetch_and_deploy_gh_release "upgopher" "wanetty/upgopher" "prebuild" "latest" "/opt/upgopher" "upgopher_.*_linux_amd64.tar.gz"
chmod +x /opt/upgopher/upgopher
- Update Debian version from 12 to 13 - Replace manual download/extraction with fetch_and_deploy_gh_release function - Remove redundant curl/wget installation (already in core dependencies) - Remove unnecessary comments - Add apt clean -y to cleanup section - Modernize apt-get commands to apt - Update success message to 'Updated successfully!' - Fix asset pattern to match release files (upgopher_*_linux_amd64.tar.gz)
82e8daf to
acc439a
Compare
Scripts wich are clearly AI generated and not further revied by the Author of this PR (in terms of Coding Standards and Script Layout) may be closed without review.
✍️ Description
Adds Upgopher, a lightweight file upload/download server written in Go. Includes LXC container creation script, installation script, and web interface metadata. Default configuration runs HTTP server on port 9090 without authentication (customizable via systemd service file).
🔗 Related PR / Issue
Link: N/A (New script submission for testing)
✅ Prerequisites (X in brackets)
🛠️ Type of Change (X in brackets)
README,AppName.md,CONTRIBUTING.md, or other docs.🔍 Code & Security Review (X in brackets)
Code_Audit.md&CONTRIBUTING.mdguidelinesAppName.sh,AppName-install.sh,AppName.json)📋 Additional Information (optional)
Source Project: https://github.com/wanetty/upgopher
Testing Results:
Customization: Users can edit
/etc/systemd/system/upgopher.serviceto configure authentication, HTTPS/TLS, port, and upload directory as detailed in the JSON notes.