Filter ga_standards by saved selection, with ?all=true bypass

Đang mở
  • Có gì không ổn?
#1,083 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
68/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python, typescript
Lĩnh vực
api, backend, frontend

Hướng nghiên cứu

Bắt đầu với entry point ga_standards trong application/web/web_main.py, sau đó kiểm tra application/frontend/src/pages/GapAnalysis/GapAnalysis.tsx và pattern ResourceSelector ?all=true hiện có. Theo dõi cách /standards áp dụng apply_user_resource_filter đồng thời duy trì việc kiểm tra tính đủ điều kiện của GA. Hoàn tất có nghĩa là các kịch bản đã liệt kê gồm người dùng đã đăng nhập, bypass, người dùng ẩn danh và flags tắt đều đạt, đồng thời Map Analysis mặc định sử dụng lựa chọn đã lưu và hỗ trợ tất cả các standards đủ điều kiện.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

enhancement not ready for development P2 untriaged

Parent: #1079 · Splits #1043 §2 · Project: https://github.com/orgs/OWASP/projects/60

What

GET /rest/v1/ga_standards feeds Map Analysis / gap-analysis dropdowns. It is not passed through apply_user_resource_filter. Completing the profile view means:

  1. Filter ga_standards the same way as /standards when the user has a non-empty selection (keep OpenCRE; empty selection = full eligible list).
  2. Frontend must pass ?all=true wherever GA should still span all eligible standards (same pattern as ResourceSelector).

Left out of #586 on purpose so we would not silently shrink the dropdowns.

Do

  • Apply apply_user_resource_filter (or equivalent) on /rest/v1/ga_standards
  • Keep GA eligibility gating (resource_name_ga_eligible_in_db)
  • Map Analysis UI: default to filtered list when a selection exists; explicit “all eligible standards” uses ?all=true
  • Tests: logged-in filtered, ?all=true bypass, anonymous full, flags off = full

Success

A user who saved ASVS+CWE sees those (if GA-eligible) in the dropdowns by default, and can still run a comparison against the full eligible set without clearing their profile.

Related

  • application/web/web_main.py ga_standards
  • application/frontend/src/pages/GapAnalysis/GapAnalysis.tsx
  • #1043
Ngôn ngữ chính
Python
Star
210
Fork
167
Merge trung bình
4 ngày 12 giờ
Pull request đã merge (30 ngày)
28

Chuẩn bị môi trường

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của OWASP/OpenCRE

Tất cả issue của OWASP/OpenCRE

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.