sam
(Sam Saffron)
6월 2, 2022, 1:07오전
23
확실히 그렇게 해야 합니다. 앞으로 며칠 내에 이 문제를 수정하겠습니다.
8개의 좋아요
riking
(Kane York)
6월 2, 2022, 11:08오후
25
여기서 문제가 된 것은 스태프가 단일 사용자에게 다중 사용 초대를 보낸 것이었으므로, 다중 사용 초대에는 자동 승인을 비활성화하고 단일 사용 초대에는 유지함으로써 기존 동작을 유지할 수 있습니다.
또한, 교육 문구(“이 사용자는 초대를 수락하면 승인됩니다”)(이제는 단일 사용 초대에만 적용)는 사이트 설정 페이지가 아니라 초대 대화상자에 표시되어야 합니다.
1개의 좋아요
sam
(Sam Saffron)
6월 2, 2022, 11:32오후
26
여기서 must_approve_users가 명시적 승인에 대한 매우 엄격한(VERY HARD LINE) 정의를 의미한다고 판단해야 할 것 같습니다.
(제가 처음에 승인했던) 묵시적 승인 방식의 문제는 엣지 케이스가 너무 많다는 점입니다. 엣지 케이스는 보안 문제와 시스템 결함을 낳습니다. 또한, 묵시적 승인에 관한 엣지 케이스를 설명하는 것은 지나치게 복잡하며, 우리가 감당할 필요가 없는 골칫거리입니다.
must_approve_users를 선택한다면, 가장 엄격한 정의를 적용하여 초대 여부나 관계없이 모든 계정에 대해 명시적으로 승인 버튼을 클릭하도록 요구할 것입니다.
8개의 좋아요
Wall-E
(Raphael)
6월 2, 2022, 11:54오후
27
정리하자면, 초대 링크는 특정 개인이 아니라, 참여 권한이 있는 여러 명이 있는 회의용 채팅방에 보내진 것이었습니다. 우리는 최대 사용 횟수를 해당 채팅방의 인원수로 설정했습니다. 그중 한 명이 이 링크를 권한이 없는 다른 조직에 속한 사람에게 전달했고, 그 사람이 채팅방의 사람들보다 먼저 링크를 사용했습니다.
3개의 좋아요
sam
(Sam Saffron)
6월 3, 2022, 5:39오전
28
참고:
committed 03:43AM - 03 Jun 22 UTC
When a site has `SiteSetting.invite_only` enabled, we create a
`ReviewableUser`… record when activating a user if the user is not
approved. Therefore, we need to approve the user when redeeming an
invite.
There are some uncertainties surrounding why a `ReviewableRecord` is
created for a user in an invites only site but this commit does not seek
to address that.
Follow-up to 7c4e2d33fa4b922354c177ffc880a2f2701a91f9
그리고
committed 02:10PM - 02 Jun 22 UTC
This security fix affects sites which have `SiteSetting.must_approve_users`
ena… bled. There are intentional and unintentional cases where invited
users can be auto approved and are deemed to have skipped the staff approval process.
Instead of trying to reason about when auto-approval should happen, we have decided that
enabling the `must_approve_users` setting going forward will just mean that all new users
must be explicitly approved by a staff user in the review queue. The only case where users are auto
approved is when the `auto_approve_email_domains` site setting is used.
Co-authored-by: Alan Guo Xiang Tan <gxtan1990@gmail.com>
이제 완료되었습니다.
@Wall-E 최신 수정 사항을 적용하기 위해 재빌드해도 좋습니다.
3개의 좋아요
Wall-E
(Raphael)
6월 3, 2022, 1:09오후
29
좋아요! 제 시스템 관리자가 인스턴스 업데이트를 담당합니다. 그는 새 버전이 여기에 올라올 때만 베타 릴리스에서 업데이트합니다:
결국 여기에 올라올까요? 그렇다면 언제쯤 될 수 있을까요?
[수정] 여기에 새 버전이 있는 것을 확인했습니다:
이것이 그 버전인가요?
1개의 좋아요
네, 먼저 해당 항목을 업데이트한 다음, 완료되면 모두 업데이트 버튼을 사용하여 나머지 모든 항목을 업그레이드해야 합니다. 커맨드라인에서 업그레이드하는 경우가 아니라면, Docker는 별도로 먼저 업그레이드해야 합니다.
3개의 좋아요
Wall-E
(Raphael)
6월 3, 2022, 6:43오후
31
샘, 이 문제를 해결해 주셔서 정말 감사합니다. 시스템 관리자가 업데이트를 완료하는 데 며칠 정도 걸릴 예정입니다.
3개의 좋아요
sam
(Sam Saffron)
6월 3, 2022, 9:17오후
32
문제가 아니에요!
@tgxworld / @martin / @gerhard 에게 감사의 말을 전하고 싶습니다. 의외로 복잡한 변경 사항이었거든요.
5개의 좋아요
sam
(Sam Saffron)
에 닫힘
6월 10, 2022, 5:40오전
33
이 주제는 7일 후 자동으로 닫혔습니다. 더 이상 답변이 불가능합니다.