toolshed/abra
4
4
Fork 23
Code Issues 82 Pull Requests 12 Packages Releases 45 Activity

fix: Allows multiple protocols on one port #741

Merged
decentral1se merged 2 commits from p4u1/abra:patch-docker-yolo into main 2026-02-13 09:29:02 +00:00
Conversation 5 Commits 2 Files Changed 21
+263 -205
p4u1 commented 2025-12-30 14:14:40 +00:00
Owner
Copy Link
Copy Source

Docs PR: toolshed/docs.coopcloud.tech#297

Docs PR: https://git.coopcloud.tech/toolshed/docs.coopcloud.tech/pulls/297
🎉 1
p4u1 added 1 commit 2025-12-30 14:14:41 +00:00
fix: Allows multiple protocols on one port
continuous-integration/drone/pr Build is failing
Details
3e98d8797f
p4u1 force-pushed patch-docker-yolo from 3e98d8797f to 0dc98051e7 2026-02-02 13:05:04 +00:00 Compare
p4u1 commented 2026-02-02 13:07:24 +00:00
Author
Owner
Copy Link
Copy Source

@decentral1se @ammaratef45 I added an integration test. What do you think? Shall we merge this? I can also clean up the test by not using my forkof the test recipe for it

@decentral1se @ammaratef45 I added an integration test. What do you think? Shall we merge this? I can also clean up the test by not using my forkof the test recipe for it
👀 1
decentral1se commented 2026-02-02 14:36:12 +00:00
Owner
Copy Link
Copy Source

@p4u1 I'm into it but I fear this might cause even more chaos with #728. I struggled to upgrade the dependencies recently due to docker upstream maintenance chaos 😬 Is there any way to create this patch somehow "locally" that it can be applied to the vendored modules? Otherwise, this creates a circle where we need to update your fork, then update our deps 👁️ Curious how you think we can manage this.

@p4u1 I'm into it but I fear this might cause even more chaos with https://git.coopcloud.tech/toolshed/abra/pulls/728. I struggled to upgrade the dependencies recently due to docker upstream maintenance chaos 😬 Is there any way to create this patch somehow "locally" that it can be applied to the vendored modules? Otherwise, this creates a circle where we need to update your fork, then update our deps 👁️ Curious how you think we can manage this.
👀 1
p4u1 commented 2026-02-08 11:21:59 +00:00
Author
Owner
Copy Link
Copy Source

@decentral1se I would also prefer a local gut patch but unfortunately this is not possible in go as far as I know.
I could move my fork to the toolshed org. Would that make it better?

@decentral1se I would also prefer a local gut patch but unfortunately this is not possible in go as far as I know. I could move my fork to the toolshed org. Would that make it better?
👍 1
p4u1 added 1 commit 2026-02-13 09:07:51 +00:00
moved fork 3bcc08f99a
p4u1 force-pushed patch-docker-yolo from 3bcc08f99a to 0effca1dd5 2026-02-13 09:08:07 +00:00 Compare
p4u1 changed title from WIP fix: Allows multiple protocols on one port to fix: Allows multiple protocols on one port 2026-02-13 09:15:02 +00:00
p4u1 commented 2026-02-13 09:15:49 +00:00
Author
Owner
Copy Link
Copy Source

@decentral1se @ammaratef45 This is ready to go from my side. I added a docs pr to document the docker/cli fork

@decentral1se @ammaratef45 This is ready to go from my side. I added a docs pr to document the docker/cli fork
decentral1se approved these changes 2026-02-13 09:27:45 +00:00
decentral1se left a comment
Owner
Copy Link
Copy Source

I'm scared at the dependency puzzle maintenance acrobatics this can bring but let's do ittttt... thanks for the work @p4u1! all good on my side to merge it in then...

I'm scared at the dependency puzzle maintenance acrobatics this can bring but let's do ittttt... thanks for the work @p4u1! all good on my side to merge it in then...
❤️ 1
decentral1se merged commit d60d426752 into main 2026-02-13 09:29:02 +00:00
decentral1se added this to the Abra v0.13 project 2026-02-15 13:49:29 +00:00
decentral1se moved this to Done in Abra v0.13 on 2026-02-15 13:49:33 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
decentral1se
Labels
Clear labels
bug
Something is not working
 build
go build related issues
 ci/cd
Building things with CI/CD
 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
 easy-first-issue
Something for new people to get stuck into. We hope it's easy!
 enhancement
New feature
 help wanted
Need some help
 i10n
Everything to do with localisation
 i18n
Everything to do with internationalisation
 installer
Everything to do with the install script.
 invalid
Something is wrong
 question
More information is needed
 release
Release management
 release-candidate
Related to the new release candidate
 security
Security related
 tech-debt
test
Unit/integration testing
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Abra v0.13
Done
Assignees
Clear assignees
3wordchant aadil (Aadil Ayub) abra-bot (Abra Bot) ammaratef45 amras (Sarma) Apfelwurm BornDeleuze Brooke carla cas (Cassowary) coopcloud cyrnel decentral1se (d1) dede devydave fauno (fauno) iexos jade (Jade Ambrose) jjsfunhouse jmakdah2 (Jackie Makdah) joe-irving (Joe Irving) kawaiipunk (KawaiiPunk) knoflook kolaente lambdabundesverband linnealovespie (April) moosemower moritz notplants oxaliq (sorrel) p4u1 pharaohgraphy (Andrew 🐦‍🔥❤️‍🔥✴️) renovate-bot (Comrade Renovate Bot) ripclap simon sixsmith (Sixsmith) stevensting trav (Trav Fryer) val (val (he/him)) yksflip
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: toolshed/abra#741