Merge pull request #559 from QN-zhangzhuo/QRTC-1984

QRTC-1984 rtc v3 update app bug fix
qiniu · Oct 24, 2022 · 90e14da · 90e14da
2 parents ae2a7af + eaa24be
commit 90e14da
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/src/main/java/com/qiniu/rtc/model/AppParam.java b/src/main/java/com/qiniu/rtc/model/AppParam.java
@@ -4,6 +4,7 @@
 
 @Data
 public class AppParam {
+    private String appId;
     private String hub;
     private String title;
     private int maxUsers;

diff --git a/src/main/java/com/qiniu/rtc/service/AppService.java b/src/main/java/com/qiniu/rtc/service/AppService.java
@@ -62,7 +62,7 @@ public Response deleteApp(String appId) throws QiniuException {
      * @throws QiniuException
      */
     public Response updateApp(AppParam appParam) throws QiniuException {
-        String urlPattern = "/v3/apps";
-        return postCall(appParam, urlPattern);
+        String urlPattern = "/v3/apps/%s";
+        return postCall(appParam, urlPattern, appParam.getAppId());
     }
 }
diff --git a/src/test/java/test/com/qiniu/rtc/RtcTest.java b/src/test/java/test/com/qiniu/rtc/RtcTest.java
@@ -29,7 +29,6 @@ public static void initQRTC() throws QiniuException {
         client = QRTC.init(TestConfig.testAccessKey, TestConfig.testSecretKey, result.getResult().getAppId());
     }
 
-
     @Disabled
     @Test
     public void testApp() throws QiniuException {
@@ -41,6 +40,7 @@ public void testApp() throws QiniuException {
         AppParam appParam = new AppParam();
         appParam.setTitle("test_rtc_example");
         appParam.setMaxUsers(15);
+        appParam.setAppId(result.getResult().getAppId());
         result = client.updateApp(appParam);
         assert result.getCode() == 200;
     }