microsoftgraph / microsoftgraph/msgraph-sdk-java
PATCH /subscriptions/{id} shouldn't require both properties
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Java
- Star
- 444
- Fork
- 154
- Merge trung bình
- 18 giờ 28 phút
- Pull request đã merge (30 ngày)
- 4
Mô tả
Describe the bug
As per the documentation, PATCH https://graph.microsoft.com/beta/subscriptions/{id} requires only 1 property to be defined. Unfortunately, as confirmed by this customer, the Java SDK requires both properties to be specified or the request fails with this error message:
Subscription must contain either the 'expirationDateTime' or 'notificationUrl' property and its value.
More details on the bug as reported by the customer @Yusvanth are in Microsoft Graph Q&A
The feature engineers have confirmed that this issue doesn't occur on the REST API or other SDKs.
Please assist to resolve the bug.
Expected behavior
Only the expirationDateTime property is required.
How to reproduce
See the customer issue + CxP response.
SDK Version
No response
Latest version known to work for scenario above?
No response
Known Workarounds
No response
Debug output
Click to expand log
```</details>
### Configuration
_No response_
### Other information
_No response_
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.
Hướng nghiên cứu
Bắt đầu bằng cách theo dõi cách Java SDK xử lý PATCH /subscriptions/{id}, sử dụng tài liệu Microsoft Graph được liên kết và issue của khách hàng làm ngữ cảnh tái hiện. Xác minh rằng một request chỉ chứa expirationDateTime được chấp nhận, đồng thời xác nhận rằng validation hiện có hoặc behavior của request không còn yêu cầu notificationUrl.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- java
- 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