Skip to content

Commit f0d82ec

Browse files
Delia 67689 main (rdkcentral#6250)
* DELIA-67689 : Increase secmanager timeout (rdkcentral#6155) Reason for change: depending on the server response time and network time the call can take 1-3 seconds normally and sometimes even a little more. Test Procedure: None Risks: None Signed-off-by: Nikita Poltorapavlo <[email protected]> * DELIA-67689 : Fix typo in json-rpc param (rdkcentral#6164) Reason for change: As there's no c++ interface for SecManager/Watermark plugins parameter names written by hand had typos. Test Procedure: None Risks: None Signed-off-by: Nikita Poltorapavlo <[email protected]> * update changelog and api version --------- Signed-off-by: Nikita Poltorapavlo <[email protected]> Co-authored-by: Anand Kandasamy <[email protected]>
1 parent 04ce6da commit f0d82ec

File tree

3 files changed

+13
-7
lines changed

3 files changed

+13
-7
lines changed

ContentProtection/CHANGELOG.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,11 @@ All notable changes to this RDK Service will be documented in this file.
1616

1717
* For more details, refer to [versioning](https://github.com/rdkcentral/rdkservices#versioning) section under Main README.
1818

19+
## [1.1.1] - 2025-05-26
20+
### Fixed
21+
- Increase secmanager timeout
22+
- Fix typo in json-rpc param
23+
1924
## [1.1.0] - 2025-05-13
2025
### Changed
2126
- New appId parameter in openDrmSession and onWatermarkStatusChanged, opacityLevel parameter in showWatermark

ContentProtection/ContentProtection.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121

2222
#define API_VERSION_NUMBER_MAJOR 1
2323
#define API_VERSION_NUMBER_MINOR 1
24-
#define API_VERSION_NUMBER_PATCH 0
24+
#define API_VERSION_NUMBER_PATCH 1
2525

2626
namespace WPEFramework {
2727
namespace Plugin {

ContentProtection/ContentProtection.h

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -166,8 +166,8 @@ namespace Plugin {
166166
SetPlaybackSessionStateParams()
167167
{
168168
Add(_T("clientId"), &ClientId);
169-
Add(_T("sessionid"), &SessionId);
170-
Add(_T("sessionstate"), &SessionState);
169+
Add(_T("sessionId"), &SessionId);
170+
Add(_T("sessionState"), &SessionState);
171171
}
172172
Core::JSON::String ClientId;
173173
Core::JSON::DecUInt32 SessionId;
@@ -180,7 +180,7 @@ namespace Plugin {
180180
ClosePlaybackSessionParams()
181181
{
182182
Add(_T("clientId"), &ClientId);
183-
Add(_T("sessionid"), &SessionId);
183+
Add(_T("sessionId"), &SessionId);
184184
}
185185
Core::JSON::String ClientId;
186186
Core::JSON::DecUInt32 SessionId;
@@ -189,7 +189,7 @@ namespace Plugin {
189189
struct SetPlaybackSpeedStateParams : public Core::JSON::Container {
190190
SetPlaybackSpeedStateParams()
191191
{
192-
Add(_T("sessionid"), &SessionId);
192+
Add(_T("sessionId"), &SessionId);
193193
Add(_T("playbackSpeed"), &PlaybackSpeed);
194194
Add(_T("playbackPosition"), &PlaybackPosition);
195195
}
@@ -285,6 +285,7 @@ namespace Plugin {
285285

286286
private:
287287
enum { Timeout = 1000 };
288+
enum { OpenSessionTimeout = 5000 };
288289

289290
private:
290291
using JSONRPCLink = WPEFramework::JSONRPC::SmartLinkType<
@@ -355,7 +356,7 @@ namespace Plugin {
355356
out["licenseRequest"] = licenseRequest;
356357
JsonObject in;
357358
result = _parent._secManager->Invoke<JsonObject, JsonObject>(
358-
Timeout, _T("openPlaybackSession"), out, in);
359+
OpenSessionTimeout, _T("openPlaybackSession"), out, in);
359360
if (result == Core::ERROR_NONE) {
360361
if (!in["success"].Boolean()) {
361362
result = Core::ERROR_GENERAL;
@@ -416,7 +417,7 @@ namespace Plugin {
416417
JsonObject in;
417418
result = _parent._secManager->Invoke<
418419
JsonObject, JsonObject>(
419-
Timeout, _T("updatePlaybackSession"), out, in);
420+
OpenSessionTimeout, _T("updatePlaybackSession"), out, in);
420421
if (result == Core::ERROR_NONE) {
421422
if (!in["success"].Boolean()) {
422423
result = Core::ERROR_GENERAL;

0 commit comments

Comments
 (0)