macadmins / macadmins/jamf-pro-sdk-python
[Bug] Create/Update Advanced Computer Search Methods Sending Empty Criteria
Chưa có ai nhận issue này.
- 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
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
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
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.
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.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- api
- Loại issue
- Lỗi
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 38/100