fix(vote): 站方处置覆盖投票链路
提问链路上 ActiveLevel 有 7 处调用,投票链路 0 处。于是 L4(整页停用)处置某人后:
/_/domain 已经 302 到 /restricted,但 /vote/{token} 照常渲染、照常收票;被处置者
还能发新投票;探索页的「热门/全部投票」也不做处置过滤 —— 同一张页面里
「箱主消失了、他的投票还在曝光」。这是运营口径不一致,不是越权。
补齐三处,与提问链路对齐:
- vote.Page L4 → 302 /restricted(同箱主页;查询失败 fail-open,与 Pager 一致)
- vote.Submit L4 → 拒绝(页面进不去不代表直接构造 POST 也拦得住)
- VotePublish L2 起拒绝(L2 已是「收窄公开」,不该再新增公开内容)
- 探索页投票列表改用 Restrictions.ActiveLevels 批量过滤(与 GetShowInHome 同口径,
一次查询避免 N+1),功能开关过滤逻辑保持不变
新增字典 key vote.err.restrictedPublish(四语言)。
验证(本地实例):
- L4 后 /vote/{token} 302 → /restricted;撤销后恢复 200
- 探索页该投票出现次数 0(处置中)→ 2(撤销后)
- 草稿投票在 L2 下发布后状态停留 draft,撤销后同一请求变为 published
This commit is contained in:
+27
-6
@@ -94,10 +94,16 @@ func getHotVotesCached(ctx context.Context, c cache.Cache) ([]*db.Vote, error) {
|
||||
return votes, nil
|
||||
}
|
||||
|
||||
// filterVotesByOwnerFeature 过滤掉箱主已关闭投票功能的投票(开关 / 定时 / 定量收集),
|
||||
// 避免探索页出现点了打不开的死链。功能配置批量加载,避免逐条查询造成 N+1;
|
||||
// 加载失败时按原样返回,依赖表抖动不应导致探索页空白。
|
||||
func filterVotesByOwnerFeature(ctx context.Context, votes []*db.Vote) []*db.Vote {
|
||||
// filterVotesForExplore 过滤掉不该出现在探索页的投票:
|
||||
//
|
||||
// 1. 箱主已关闭投票功能(开关 / 定时 / 定量收集)—— 避免点开是死链;
|
||||
// 2. 箱主处于站方处置中 —— 与首页箱主列表(GetShowInHome 用
|
||||
// activeRestrictionsSubQuery)保持同一口径。此前只过滤了功能开关,
|
||||
// 于是同一张探索页里会出现「箱主本人在列表里消失了,他的投票还在曝光」。
|
||||
//
|
||||
// 两次配置都是批量加载,避免逐条查询造成 N+1;任一侧加载失败时按「该项不过滤」
|
||||
// 处理,依赖表抖动不应让探索页整页空白。
|
||||
func filterVotesForExplore(ctx context.Context, votes []*db.Vote) []*db.Vote {
|
||||
if len(votes) == 0 {
|
||||
return votes
|
||||
}
|
||||
@@ -111,6 +117,18 @@ func filterVotesByOwnerFeature(ctx context.Context, votes []*db.Vote) []*db.Vote
|
||||
}
|
||||
}
|
||||
|
||||
// 生效处置(任意级别):这批箱主的内容不再进入公开曝光
|
||||
restricted := make(map[uint]bool, len(ownerIDs))
|
||||
if levels, err := db.Restrictions.ActiveLevels(ctx.Request().Context(), ownerIDs); err != nil {
|
||||
logrus.WithContext(ctx.Request().Context()).WithError(err).Warn("Failed to batch load user restrictions")
|
||||
} else {
|
||||
for uid, level := range levels {
|
||||
if level != "" {
|
||||
restricted[uid] = true
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
controls, err := db.FeatureControls.ListByUserIDs(ctx.Request().Context(), ownerIDs, db.FeatureVote)
|
||||
if err != nil {
|
||||
logrus.WithContext(ctx.Request().Context()).WithError(err).Warn("Failed to batch load vote feature controls")
|
||||
@@ -120,6 +138,9 @@ func filterVotesByOwnerFeature(ctx context.Context, votes []*db.Vote) []*db.Vote
|
||||
now := time.Now()
|
||||
out := make([]*db.Vote, 0, len(votes))
|
||||
for _, v := range votes {
|
||||
if restricted[v.UserID] {
|
||||
continue
|
||||
}
|
||||
fc, ok := controls[v.UserID]
|
||||
if !ok {
|
||||
fc = db.DefaultFeatureControl(v.UserID, db.FeatureVote)
|
||||
@@ -178,14 +199,14 @@ func Explore(ctx context.Context, c cache.Cache) {
|
||||
logrus.WithContext(ctx.Request().Context()).WithError(err).Warn("Failed to get hot votes")
|
||||
hotVotes = []*db.Vote{}
|
||||
}
|
||||
ctx.Data["HotVotes"] = filterVotesByOwnerFeature(ctx, hotVotes)
|
||||
ctx.Data["HotVotes"] = filterVotesForExplore(ctx, hotVotes)
|
||||
|
||||
allVotes, err := db.Votes.ListPublishedPublic(ctx.Request().Context(), 30)
|
||||
if err != nil {
|
||||
logrus.WithContext(ctx.Request().Context()).WithError(err).Warn("Failed to get public votes for explore")
|
||||
allVotes = []*db.Vote{}
|
||||
}
|
||||
ctx.Data["AllVotes"] = filterVotesByOwnerFeature(ctx, allVotes)
|
||||
ctx.Data["AllVotes"] = filterVotesForExplore(ctx, allVotes)
|
||||
|
||||
// 计算总页数(向上取整)
|
||||
totalPages := (int(total) + pageSize - 1) / pageSize
|
||||
|
||||
@@ -915,6 +915,21 @@ func VotePublish(ctx context.Context) {
|
||||
return
|
||||
}
|
||||
|
||||
// 站方处置:L2 起不得再发布新的公开内容(与「设公开」同口径)。
|
||||
// 此前投票链路完全没有处置校验:被处置的箱主仍能发新投票并进探索页曝露。
|
||||
level, _, err := db.Restrictions.ActiveLevel(ctx.Request().Context(), ctx.User.ID)
|
||||
if err != nil {
|
||||
logrus.WithContext(ctx.Request().Context()).WithError(err).Error("Failed to query user restriction")
|
||||
ctx.SetInternalErrorFlash()
|
||||
ctx.Redirect("/user/questions#votes")
|
||||
return
|
||||
}
|
||||
if db.RestrictionAtLeast(level, db.RestrictionL2) {
|
||||
ctx.SetErrorFlash(i18n.T(ctx.Lang(), "vote.err.restrictedPublish"))
|
||||
ctx.Redirect("/user/questions#votes")
|
||||
return
|
||||
}
|
||||
|
||||
if err := validateForPublish(ctx, vote); err != nil {
|
||||
ctx.SetErrorFlash(err.Error())
|
||||
ctx.Redirect("/user/questions#votes")
|
||||
|
||||
Reference in New Issue
Block a user