Skip to content

Commit f4d8132

Browse files
authored
Merge pull request #578 from gsmet/fix-serialization
Fix issues introduced by previous GitHubEvent serialization fixes
2 parents 38df501 + 4d25461 commit f4d8132

File tree

3 files changed

+23
-15
lines changed

3 files changed

+23
-15
lines changed

deployment/src/main/resources/META-INF/resources/replay-ui/index.html

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@
9292
<div style="clear: both"></div>
9393
</div>
9494
<div class="description" v-show="gitHubEvent.showPayload">
95-
<pre>{{ gitHubEvent.parsedPayload }}</pre>
95+
<pre>{{ gitHubEvent.payload | pretty }}</pre>
9696
</div>
9797
</div>
9898
</div>
@@ -185,7 +185,7 @@
185185
return;
186186
}
187187
gitHubEventsApp.waitForEvents = false;
188-
188+
189189
let gitHubEvent = JSON.parse(message.data);
190190
if (!gitHubEventsApp.visible) {
191191
gitHubEvent.new = true;
@@ -208,7 +208,7 @@
208208
},
209209
updateEventSourceStatus: function () {
210210
localGitHubEventsApp = gitHubEventsApp ? gitHubEventsApp : this;
211-
211+
212212
if (eventSource.readyState == 0) {
213213
localGitHubEventsApp.eventSourceStatusColor = 'ui grey button small';
214214
localGitHubEventsApp.eventSourceStatusIcon = 'notched circle loading icon';
@@ -225,7 +225,7 @@
225225
console.error('could not find event for ' + deliveryId);
226226
return;
227227
}
228-
228+
229229
this.reconnectEventSource();
230230

231231
const gitHubEvent = gitHubEventsApp.gitHubEventsMap.get(deliveryId);
@@ -257,7 +257,7 @@
257257
}
258258

259259
const gitHubEvent = gitHubEventsApp.gitHubEventsMap.get(deliveryId);
260-
navigator.clipboard.writeText(JSON.stringify(gitHubEvent.parsedPayload, null, 4));
260+
navigator.clipboard.writeText(JSON.stringify(JSON.parse(gitHubEvent.payload), null, 4));
261261
},
262262
markInvisible: function() {
263263
gitHubEventsApp.visible = false;
@@ -271,6 +271,11 @@
271271
favicon.reset();
272272
gitHubEventsApp.visible = true;
273273
}
274+
},
275+
filters: {
276+
pretty: function(value) {
277+
return JSON.stringify(JSON.parse(value), null, 4);
278+
}
274279
}
275280
});
276281
</script>

docs/modules/ROOT/pages/includes/attributes.adoc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
:quarkus-version: 3.8.0
1+
:quarkus-version: 3.8.2
22
:quarkus-github-app-version: 2.3.1
33

44
:github-api-javadoc-root-url: https://github-api.kohsuke.org/apidocs/org/kohsuke/github

runtime/src/main/java/io/quarkiverse/githubapp/GitHubEvent.java

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,8 @@ public class GitHubEvent {
2323

2424
private final String action;
2525

26+
private final String eventAction;
27+
2628
private final String payload;
2729

2830
private final boolean replayed;
@@ -37,6 +39,15 @@ public GitHubEvent(Long installationId, String appName, String deliveryId, Strin
3739
this.action = action;
3840
this.payload = payload;
3941
this.replayed = replayed;
42+
43+
StringBuilder eventActionSb = new StringBuilder();
44+
if (event != null && !event.isBlank()) {
45+
eventActionSb.append(event);
46+
}
47+
if (action != null && !action.isBlank()) {
48+
eventActionSb.append(".").append(action);
49+
}
50+
this.eventAction = eventActionSb.toString();
4051
}
4152

4253
public Long getInstallationId() {
@@ -69,16 +80,8 @@ public String getAction() {
6980
return action;
7081
}
7182

72-
@JsonIgnore
7383
public String getEventAction() {
74-
StringBuilder sb = new StringBuilder();
75-
if (event != null && !event.isBlank()) {
76-
sb.append(event);
77-
}
78-
if (action != null && !action.isBlank()) {
79-
sb.append(".").append(action);
80-
}
81-
return sb.toString();
84+
return eventAction;
8285
}
8386

8487
public String getPayload() {

0 commit comments

Comments
 (0)