microsoftgraph / microsoftgraph/msgraph-sdk-java
PATCH /subscriptions/{id} shouldn't require both properties
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 444
- Forks
- 154
- Avg merge
- 18h 28m
- Merged PRs (30d)
- 4
Description
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_
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing the Java SDK handling for PATCH /subscriptions/{id}, using the linked Microsoft Graph documentation and customer issue as the reproduction context. Verify that a request containing only expirationDateTime is accepted, and confirm that the existing validation or request behavior no longer requires notificationUrl.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- api
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100