[Bug] Create/Update Advanced Computer Search Methods Sending Empty Criteria
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 38/100
Hướng nghiên cứu
Bắt đầu từ các entry point create_advanced_computer_search và update_advanced_computer_search_by_id, sau đó theo dõi cách ClassicAdvancedComputerSearch.criteria được tuần tự hóa cho request Jamf Pro. Tái hiện vấn đề với các ví dụ criteria được cung cấp và xác minh rằng việc tạo và cập nhật một search vẫn giữ nguyên các tiêu chí khi truy xuất search đó theo ID.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
tl;dr when attempting to create or update an advanced computer search (using the create_advanced_computer_search or update_advanced_computer_search_by_id methods, respectively), the operation is completed successfully, except for the search criteria. Name and display fields are set as configured, but any criteria sent are seemingly ignored, and the search is created/updated with no criteria. This effectively places every device into the results for the advanced computer search.
Reproduction details below will be for creating a search, but the result is the same when attempting to update an existing search.
Steps to Reproduce
from jamf_pro_sdk import JamfProClient, SessionConfig
from jamf_pro_sdk.models.classic import computer_groups, advanced_computer_searches
from jamf_pro_sdk.models.classic.criteria import ClassicCriterion
from jamf_pro_sdk.clients.auth import ApiClientCredentialsProvider
jamfClient = JamfProClient(
server=jamfURL,
credentials=ApiClientCredentialsProvider(jamfClientID, jamfClientSecret),
session_config=SessionConfig(**{"timeout": 30, "max_retries": 3}),
)
notUpdatedSearchData = advanced_computer_searches.ClassicAdvancedComputerSearch(
name="macOS Version Not Current",
criteria=[
ClassicCriterion(
name='Managed',
priority=0,
and_or='and',
search_type='is',
value='Managed',
opening_paren=False,
closing_paren=False
),
ClassicCriterion(
name='Operating System Version',
priority=1,
and_or='and',
search_type='less than',
value='15.1',
opening_paren=False,
closing_paren=False
)
],
display_fields=[
advanced_computer_searches.ClassicAdvancedComputerSearchDisplayField(
name="JSS Computer ID"
),
advanced_computer_searches.ClassicAdvancedComputerSearchDisplayField(
name="Last Check-in"
),
advanced_computer_searches.ClassicAdvancedComputerSearchDisplayField(
name="Serial Number"
),
advanced_computer_searches.ClassicAdvancedComputerSearchDisplayField(
name="Operating System Version"
),
advanced_computer_searches.ClassicAdvancedComputerSearchDisplayField(
name="Username"
),
],
)
searchID = jamfClient.classic_api.create_advanced_computer_search(data=notUpdatedSearchData)
searchData = jamfClient.classic_api.get_advanced_computer_search_by_id(advanced_search=searchID)
print(searchData)
Expected Result
Expected an advanced computer search to be created with the following criteria:
Actual Result
Advanced computer search created with no criteria:
id=6 name='macOS Version Not Current' site=ClassicSite(id=-1, name='None') criteria=[] display_fields=[ClassicAdvancedComputerSearchDisplayField(name='Username'), ClassicAdvancedComputerSearchDisplayField(name='Operating System Version'), ClassicAdvancedComputerSearchDisplayField(name='JSS Computer ID'), ClassicAdvancedComputerSearchDisplayField(name='Serial Number'), ClassicAdvancedComputerSearchDisplayField(name='Last Check-in')]
(omitted computers from above output)
Additional Data
If I retrieve the criteria from an existing advanced computer search (with correct criteria) and feed that directly into the create method, I would definitely expect this criteria to be valid and accepted. Even doing this, the new search is created with no criteria:
knownGoodSearchCriteria = jamfClient.classic_api.get_advanced_computer_search_by_id(advanced_search=5).criteria
print(knownGoodSearchCriteria)
####
[ClassicCriterion(name='Managed', priority=0, and_or='and', search_type='is', value='Managed', opening_paren=False, closing_paren=False), ClassicCriterion(name='Operating System Version', priority=1, and_or='and', search_type='less than', value='15.1', opening_paren=False, closing_paren=False)]
####
notUpdatedSearchData = advanced_computer_searches.ClassicAdvancedComputerSearch(
name="macOS Version Not Current",
criteria=knownGoodSearchCriteria,
display_fields=[
advanced_computer_searches.ClassicAdvancedComputerSearchDisplayField(
name="JSS Computer ID"
),
advanced_computer_searches.ClassicAdvancedComputerSearchDisplayField(
name="Last Check-in"
),
advanced_computer_searches.ClassicAdvancedComputerSearchDisplayField(
name="Serial Number"
),
advanced_computer_searches.ClassicAdvancedComputerSearchDisplayField(
name="Operating System Version"
),
advanced_computer_searches.ClassicAdvancedComputerSearchDisplayField(
name="Username"
),
],
)
searchID = jamfClient.classic_api.create_advanced_computer_search(data=notUpdatedSearchData)
searchData = jamfClient.classic_api.get_advanced_computer_search_by_id(advanced_search=searchID)
print(searchData)
####
id=7 name='macOS Version Not Current' site=ClassicSite(id=-1, name='None') criteria=[] display_fields=[ClassicAdvancedComputerSearchDisplayField(name='Username'), ClassicAdvancedComputerSearchDisplayField(name='JSS Computer ID'), ClassicAdvancedComputerSearchDisplayField(name='Last Check-in'), ClassicAdvancedComputerSearchDisplayField(name='Serial Number'), ClassicAdvancedComputerSearchDisplayField(name='Operating System Version')]
####
(again omitted computers)
System Information
macOS 15.1
Python 3.12.1
SDK 0.6a2
Jamf Pro 11.10.2
- Ngôn ngữ chính
- Python
- Star
- 69
- Fork
- 16
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
Chúng tôi chưa kiểm tra các tệp thiết lập môi trường của dự án này. Hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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 macadmins/jamf-pro-sdk-python
-
enhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
-
[Bug] preview/mdm/commands deprecated & removed which breaks send_mdm_command_preview() method.Có thể làm lại được @liquidz00 đã nhận 409 ngày trước và không có pull request nào đang mở. Đang mởbug
macadmins/jamf-pro-sdk-python#68 · 1 bình luận · 1 người được giao ·
-
[Feature Request] Pass a model to the Classic and Pro API request methods as the return type.Có thể làm lại được @liquidz00 đã nhận 463 ngày trước và không có pull request nào đang mở. Đang mởenhancement
macadmins/jamf-pro-sdk-python#52 · 1 bình luận · 2 người được giao ·
-
[Feature Request] Option to parse XML from Classic API endpointsCó thể làm lại được @brysontyrrell đã nhận 1033 ngày trước và không có pull request nào đang mở. Đang mởenhancement feedback
macadmins/jamf-pro-sdk-python#33 · 4 bình luận · 1 người được giao ·
-
feedback
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
macadmins/jamf-pro-sdk-python#30 · 1 bình luận ·
Tất cả issue của macadmins/jamf-pro-sdk-python
Issue tương tự
-
correction metadata
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
acl-org/acl-anthology#10104 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug status/needs-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
prowler-cloud/prowler#12885 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Bug in GaussianTailProbabilityCalibrator: running_statistics=False still uses a windowed varianceĐang mởbug good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
selimfirat/pysad#107 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug ci-failure high priority
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
vllm-project/vllm-omni#8194 · 1 bình luận ·
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 88/100
Maintainer thường phản hồi trong vòng 1 ngày