Skip to content
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

UDS: Make all remote addr 0.0.0.0 #4390

Merged
merged 6 commits into from
Feb 13, 2025
Merged

UDS: Make all remote addr 0.0.0.0 #4390

merged 6 commits into from
Feb 13, 2025

Conversation

Fangliding
Copy link
Member

需要验证 是否正常工作

@RPRX
Copy link
Member

RPRX commented Feb 13, 2025

我发现直接改 combinedListener 就行了,这东西就 uds 在用

@Fangliding
Copy link
Member Author

Fangliding commented Feb 13, 2025

review在TG看不到消息所以没回(

accept我觉得只要返回一个包装conn就行了 其实err检查都没必要 该怎么处理怎么处理 wrapper 而且 干完该干的其余保持原逻辑透传就是了 我无聊写的
combinedListener我是没看 改这里也行

@RPRX
Copy link
Member

RPRX commented Feb 13, 2025

那样的话返回的 conn 不为 nil,不好,Go 项目的逻辑几乎全是如果有 err 那么第一个值就为 nil

@RPRX RPRX merged commit 22c50a7 into main Feb 13, 2025
70 checks passed
@RPRX RPRX deleted the UDS branch February 13, 2025 14:02
@RPRX
Copy link
Member

RPRX commented Feb 13, 2025

频道评论那个广告帮忙踢了,channel 已经 remove 他了

@RPRX
Copy link
Member

RPRX commented Feb 13, 2025

@Fangliding 怎么还没踢他

@Fangliding
Copy link
Member Author

啊?最近没干架吧

@RPRX
Copy link
Member

RPRX commented Feb 13, 2025

今天第一条频道消息的第二条评论是广告,每次一发频道他就来刷存在感

@Fangliding
Copy link
Member Author

够隐蔽的 sticker 又看不到他ID 不点开还真不知道 如果其他人看到会report的 证明其他人也没看到((

@RPRX
Copy link
Member

RPRX commented Feb 13, 2025

感觉是故意发个哭泣为底的表情包,引得 bot 回复他,就看到他昵称了

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.

2 participants