A news link can't be cleared, and settings can't be saved without one
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 76/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- typescript
- Lĩnh vực
- backend
Hướng nghiên cứu
Bắt đầu bằng cách lần theo handler PATCH /settings và phần xác thực newsLink, đặc biệt là lời gọi sanitiseAndValidateUrl; kiểm tra mã của trang cài đặt gửi newsLink dưới dạng null khi giá trị này để trống. Xem các bài kiểm tra xác thực hiện có và bổ sung kiểm thử cho giá trị trống và không trống. Hoàn thành khi newsLink trống được chấp nhận dưới dạng null, các giá trị không trống vẫn được kiểm tra và có thể lưu cài đặt mà không cần liên kết tin tức.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
PATCH /settings passes every newsLink it receives through sanitiseAndValidateUrl, and @braintree/sanitize-url turns null and '' into about:blank, which fails urlRegex, so the request gets a 422. The settings page always sends newsLink, as null when the field is empty. So an admin can't clear the news link once it's set, and an organisation without one (the schema default) can't save any setting from that page.
The participant portal treats "no news link" as valid (#598): the menu hides News, and /news redirects home. Clearing worked until #862 added the check for #860.
Fix: accept an empty newsLink as null, and only check non-empty values.
Related: #598, #727, #764, #860, #862.
- Ngôn ngữ chính
- TypeScript
- Star
- 4
- Fork
- 0
- Merge trung bình
- 14 giờ 50 phút
- Pull request đã merge (30 ngày)
- 5
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của Garvan-Data-Science-Platform/ctrl
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 82/100
-
backend bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Garvan-Data-Science-Platform/ctrl#970 · 1 bình luận ·
-
backend bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
backend bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
backend bug enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Garvan-Data-Science-Platform/ctrl#956 · 1 bình luận ·
Tất cả issue của Garvan-Data-Science-Platform/ctrl
Issue tương tự
-
Bump anybuf version to ^1.0.0Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
wormhole-foundation/wormhole#5062 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
internal os specific
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
frostney/GocciaScript#1688 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
integration:hermes
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
vectorize-io/hindsight#5592 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
vercel/streamdown#673 · 1 reaction ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
apache/rocketmq-dashboard#6712 ·
Maintainer thường phản hồi trong vòng 3 ngày