toolshed/abra
toolshed/abra
4
2
Fork 18
Code Issues 29 Pull Requests 3 Packages Releases 40 Activity

fix: sshPkg.Fatal has more nuance #525

Merged
decentral1se merged 1 commits from server-add-error-handler into main 2025-03-23 09:30:26 +00:00
Conversation 0 Commits 1 Files Changed 2 +3 -2
decentral1se commented 2025-03-23 09:29:59 +00:00
Owner
Copy Link

See #507

It's not always "just ssh to the host", it can be something from docker on the remote side. So, it seems best to defer to the mapping of errors in sshPkg.Fatal and improve those. They're more specific to the actual problem with a 1-to-1 mapping.

Hopefully we can improve the overall error reporting / hinting situation soon.

See https://git.coopcloud.tech/toolshed/abra/issues/507 It's not always "just ssh to the host", it can be something from docker on the remote side. So, it seems best to defer to the mapping of errors in `sshPkg.Fatal` and improve those. They're more specific to the actual problem with a 1-to-1 mapping. Hopefully we can improve the overall error reporting / hinting situation soon.
decentral1se added 1 commit 2025-03-23 09:29:59 +00:00
fix: sshPkg.Fatal has more nuance
All checks were successful
continuous-integration/drone/pr Build is passing
Details
continuous-integration/drone/push Build is passing
Details
6d64e0edd3 
See #507
decentral1se merged commit 6d64e0edd3 into main 2025-03-23 09:30:26 +00:00
decentral1se deleted branch server-add-error-handler 2025-03-23 09:30:26 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
build
go build related issues
ci/cd
Building things with CI/CD
contributing
Dealing with contributing
critical fix
https://docs.coopcloud.tech/federation/resolutions/passed/010/
design
UI/UX
documentation
Documenting all the things
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
meta
relating to the project or repo itself
question
More information is needed
release
Release management
release-candidate
Related to the new release candidate
security
Security related
test
Unit/integration testing
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
3wordchant abra-bot ammaratef45 Apfelwurm carla cas decentral1se fauno kawaiipunk knoflook lambdabundesverband moritz p4u1 simon trav val yksflip
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: toolshed/abra#525
No description provided.
Delete Branch "server-add-error-handler"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?