highperfocused/nak
1
0
Fork 0
mirror of https://github.com/fiatjaf/nak.git synced 2026-04-13 00:46:54 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #118 from fiatjaf/fix/event-split-safety

Browse Source 
fix: use strings.Cut instead of strings.Split to avoid potential panic
This commit is contained in:
mattn
2026-03-30 14:10:25 +09:00
committed by GitHub
parent 0dbe14aa93 3d650ee502
commit de599c64f6
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
event.go
View File

@@ -354,7 +354,8 @@ func publishFlow(ctx context.Context, c *cli.Command, kr nostr.Signer, evt nostr
if c.Bool("confirm") {
relaysStr := make([]string, len(relays))
for i, r := range relays {
relaysStr[i] = strings.ToLower(strings.Split(r.URL, "://")[1])
_, after, _ := strings.Cut(r.URL, "://")
relaysStr[i] = strings.ToLower(after)
}
time.Sleep(time.Millisecond * 10)
if !askConfirmation("publish to [ " + strings.Join(relaysStr, " ") + " ]? ") {