ra/runelite
1
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

Merge pull request #408 from UniquePassive/fix-empty-config-value

Browse Source 
http-service: fix exception on empty config setKey request body
This commit is contained in:
Adam
2018-01-23 10:07:17 -05:00
committed by GitHub
parent 8a3e11e4f7 741d42ab49
commit 5bd1cf4759
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
http-service/src/main/java/net/runelite/http/service/config/ConfigService.java
View File

@@ -114,7 +114,7 @@ public class ConfigService
HttpServletRequest request, HttpServletRequest request,
HttpServletResponse response, HttpServletResponse response,
@PathVariable String key, @PathVariable String key,
@RequestBody String value @RequestBody(required = false) String value
) throws IOException ) throws IOException
{ {
SessionEntry session = auth.handle(request, response); SessionEntry session = auth.handle(request, response);
@@ -129,7 +129,7 @@ public class ConfigService
con.createQuery("insert into config (user, `key`, value) values (:user, :key, :value) on duplicate key update value = :value") con.createQuery("insert into config (user, `key`, value) values (:user, :key, :value) on duplicate key update value = :value")
.addParameter("user", session.getUser()) .addParameter("user", session.getUser())
.addParameter("key", key) .addParameter("key", key)
.addParameter("value", value) .addParameter("value", value != null ? value : "")
.executeUpdate(); .executeUpdate();
} }
} }