Conversation
*/ | ||
@OPTIONS | ||
public Response crossOriginForInstallations(@Context HttpHeaders headers) { | ||
System.out.println("IN OPTIONS"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
syso :)
Conflicts: src/main/java/org/jboss/aerogear/aerodoc/rest/Login.java
public class CorsExceptionHandler implements ExceptionMapper<EJBException> { | ||
@Override | ||
public Response toResponse(final EJBException exception) { | ||
System.out.println("Handling Unauthorized exception"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
syso :)
since the 0.2.1 JAR of the java-client is not present, the build will fail Can you make travis happy, similar to here: (make sure you use the correct brnach) |
…ll receive messages again
@@ -48,11 +49,13 @@ public LeadSender() { | |||
public void sendLeads(List<String> users, Lead lead) { | |||
|
|||
Map categories = new HashMap(); | |||
|
|||
categories.put("lead", "version=" + leadVersion++); //TODO manage the version properly | |||
UnifiedMessage unifiedMessage = new UnifiedMessage.Builder() | |||
.pushApplicationId(getActivePushConfig().getPushApplicationId()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what if there is no (active) PushConfig ? This will throw a NPE
👍 |
landed in ba24eea |
Again, that landed message was unnecessary. Especially since this PR landed multiple commits |
No description provided.