-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Introduce
Report an Issue
menu in DevUI
- This opens up the GitHub issue with some prefilled data - Fixes #32431
- Loading branch information
Showing
3 changed files
with
206 additions
and
0 deletions.
There are no files selected for viewing
28 changes: 28 additions & 0 deletions
28
...rc/main/java/io/quarkus/vertx/http/deployment/devmode/console/ReportAnIssueProcessor.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
package io.quarkus.vertx.http.deployment.devmode.console; | ||
|
||
import io.quarkus.deployment.IsDevelopment; | ||
import io.quarkus.deployment.annotations.BuildStep; | ||
import io.quarkus.devui.deployment.InternalPageBuildItem; | ||
import io.quarkus.devui.runtime.ReportAnIssueJsonRPCService; | ||
import io.quarkus.devui.spi.JsonRPCProvidersBuildItem; | ||
import io.quarkus.devui.spi.page.Page; | ||
|
||
public class ReportAnIssueProcessor { | ||
@BuildStep(onlyIf = IsDevelopment.class) | ||
JsonRPCProvidersBuildItem registerJsonRpcService() { | ||
return new JsonRPCProvidersBuildItem("report-an-issue", ReportAnIssueJsonRPCService.class); | ||
} | ||
|
||
@BuildStep(onlyIf = IsDevelopment.class) | ||
InternalPageBuildItem createBuildMetricsPages() { | ||
InternalPageBuildItem item = new InternalPageBuildItem("Report an Issue", 99); | ||
|
||
item.addPage(Page.webComponentPageBuilder().internal() | ||
.namespace("devui-report-issues") | ||
.title("Report an issue") | ||
.icon("font-awesome-solid:bug") | ||
.componentLink("qwc-report-issues.js")); | ||
|
||
return item; | ||
} | ||
} |
44 changes: 44 additions & 0 deletions
44
extensions/vertx-http/dev-ui-resources/src/main/resources/dev-ui/qwc/qwc-report-issues.js
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,44 @@ | ||
import { LitElement, html, css} from 'lit'; | ||
import { JsonRpc } from 'jsonrpc'; | ||
import '@vaadin/button'; | ||
|
||
/** | ||
* This component shows the Report Issues Page | ||
*/ | ||
export class QwcReportIssues extends LitElement { | ||
|
||
jsonRpc = new JsonRpc(this); | ||
|
||
static styles = css` | ||
.todo { | ||
padding-left: 10px; | ||
height: 100%; | ||
}`; | ||
|
||
constructor() { | ||
super(); | ||
} | ||
|
||
render() { | ||
return html` | ||
<vaadin-horizontal-layout theme="spacing" style="align-items: baseline"> | ||
<vaadin-button @click="${this._reportBug}">Report a bug in Quarkus</vaadin-button> | ||
<vaadin-button @click="${this._reportFeature}">Request a new feature/enhancement</vaadin-button> | ||
</vaadin-horizontal-layout> | ||
`; | ||
} | ||
|
||
_reportBug(event){ | ||
event.preventDefault(); | ||
this.jsonRpc.reportBug().then(e => { | ||
window.open(e.result.url, "_blank"); | ||
}); | ||
} | ||
|
||
_reportFeature(event) { | ||
event.preventDefault(); | ||
window.open("https://github.com/quarkusio/quarkus/issues/new?assignees=&labels=kind%2Fenhancement&template=feature_request.yml", "_blank"); | ||
} | ||
|
||
} | ||
customElements.define('qwc-report-issues', QwcReportIssues); |
134 changes: 134 additions & 0 deletions
134
...ertx-http/runtime/src/main/java/io/quarkus/devui/runtime/ReportAnIssueJsonRPCService.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,134 @@ | ||
package io.quarkus.devui.runtime; | ||
|
||
import java.io.BufferedReader; | ||
import java.io.File; | ||
import java.io.InputStream; | ||
import java.io.InputStreamReader; | ||
import java.io.UnsupportedEncodingException; | ||
import java.net.URL; | ||
import java.net.URLEncoder; | ||
import java.nio.charset.StandardCharsets; | ||
import java.util.Arrays; | ||
import java.util.Map; | ||
import java.util.Objects; | ||
|
||
import jakarta.enterprise.context.ApplicationScoped; | ||
|
||
import io.quarkus.logging.Log; | ||
import io.smallrye.common.os.OS; | ||
import io.vertx.core.json.JsonObject; | ||
|
||
@ApplicationScoped | ||
public class ReportAnIssueJsonRPCService { | ||
|
||
public JsonObject reportBug() { | ||
URLBuilder urlBuilder = new URLBuilder( | ||
"https://github.com/quarkusio/quarkus/issues/new?assignees=&labels=kind%2Fbug&template=bug_report.yml"); | ||
gatherInfo(urlBuilder); | ||
return new JsonObject( | ||
Map.of("url", urlBuilder.toString())); | ||
} | ||
|
||
String run(String... command) { | ||
Process process = null; | ||
StringBuilder responseBuilder = new StringBuilder(); | ||
try { | ||
ProcessBuilder processBuilder = new ProcessBuilder().command(command); | ||
|
||
process = processBuilder.start(); | ||
try (InputStream inputStream = process.getInputStream()) { | ||
try (BufferedReader reader = new BufferedReader(new InputStreamReader(inputStream, StandardCharsets.UTF_8))) { | ||
for (String line = reader.readLine(); line != null; line = reader.readLine()) { | ||
responseBuilder.append(line); | ||
} | ||
safeWaitFor(process); | ||
} | ||
} catch (Throwable t) { | ||
safeWaitFor(process); | ||
throw t; | ||
} | ||
} catch (Exception e) { | ||
Log.warn("Error while running command: " + Arrays.toString(command), e); | ||
return ""; | ||
} finally { | ||
if (process != null && process.isAlive()) { | ||
process.destroy(); | ||
} | ||
} | ||
return responseBuilder.toString(); | ||
} | ||
|
||
private static void safeWaitFor(Process process) { | ||
boolean intr = false; | ||
try { | ||
for (;;) | ||
try { | ||
process.waitFor(); | ||
return; | ||
} catch (InterruptedException ex) { | ||
intr = true; | ||
} | ||
} finally { | ||
if (intr) { | ||
Thread.currentThread().interrupt(); | ||
} | ||
} | ||
} | ||
|
||
void gatherInfo(URLBuilder builder) { | ||
builder.addQueryParameter("java_version", System.getProperty("java.version")) | ||
.addQueryParameter("quarkus_version", | ||
Objects.toString(getClass().getPackage().getImplementationVersion(), "999-SNAPSHOT")); | ||
|
||
if (OS.WINDOWS.isCurrent()) { | ||
builder.addQueryParameter("uname", run("cmd.exe", "/C", "ver")); | ||
if (new File("mvnw.cmd").exists()) { | ||
builder.addQueryParameter("build_tool", run("./mvnw.cmd", "--version")); | ||
} | ||
if (new File("gradlew.bat").exists()) { | ||
builder.addQueryParameter("build_tool", run("./gradlew.bat", "--version")); | ||
} | ||
} else { | ||
builder.addQueryParameter("uname", run("uname", "-a")); | ||
if (new File("mvnw").exists()) { | ||
builder.addQueryParameter("build_tool", run("./mvnw", "--version")); | ||
} | ||
if (new File("gradlew").exists()) { | ||
builder.addQueryParameter("build_tool", run("./gradlew", "--version")); | ||
} | ||
} | ||
} | ||
|
||
static class URLBuilder { | ||
private final StringBuilder url; | ||
|
||
public URLBuilder(String url) { | ||
this.url = new StringBuilder(url); | ||
} | ||
|
||
public URLBuilder addQueryParameter(String key, String value) { | ||
if (this.url.indexOf("?") == -1) { | ||
this.url.append("?"); | ||
} else { | ||
this.url.append("&"); | ||
} | ||
|
||
this.url.append(encodeToUTF(key).replaceAll("[+]", "%20")).append("=") | ||
.append(encodeToUTF(value.replaceAll(System.lineSeparator(), " ")).replaceAll("[+]", "%20")); | ||
return this; | ||
} | ||
|
||
static String encodeToUTF(String value) { | ||
try { | ||
return URLEncoder.encode(value, StandardCharsets.UTF_8.displayName()); | ||
} catch (UnsupportedEncodingException var2) { | ||
throw new RuntimeException(var2); | ||
} | ||
} | ||
|
||
public String toString() { | ||
return this.url.toString(); | ||
} | ||
} | ||
|
||
} |