Hipchat notifications failed if room name contains spaces

[b-viguier]

Hello,
Since the migration to Hipchat API v2, the room id is contained in the URL, and must be url-encoded (you can check corresponding documentation here. Unfortunately, the hipchat notification service does not:

seyren/seyren-core/src/main/java/com/seyren/core/service/notification/HipChatNotificationService.java

Line 89 in e40cdd0

String url = baseUrl + "/v2/room/" + roomId + "/notification?auth_token=" + authToken;

Workaraound: As a Seyren user, you can rename your room, or use the numerical id of the room… but if you have lot of checks to update it may be painful 😢

Thanks for your work!
Benoit