From 79a2d00eb647a038676300da6cc5d254465755ec Mon Sep 17 00:00:00 2001 From: Sebastian Stenzel Date: Fri, 16 Jul 2021 10:11:14 +0200 Subject: [PATCH 1/5] adjusted bug report templates [ci skip] --- .github/ISSUE_TEMPLATE/bug.md | 56 ------------------ .github/ISSUE_TEMPLATE/bug.yml | 93 ++++++++++++++++++++++++++++++ .github/ISSUE_TEMPLATE/feature.md | 27 --------- .github/ISSUE_TEMPLATE/feature.yml | 37 ++++++++++++ .github/workflows/triageBugs.yml | 37 ------------ 5 files changed, 130 insertions(+), 120 deletions(-) delete mode 100644 .github/ISSUE_TEMPLATE/bug.md create mode 100644 .github/ISSUE_TEMPLATE/bug.yml delete mode 100644 .github/ISSUE_TEMPLATE/feature.md create mode 100644 .github/ISSUE_TEMPLATE/feature.yml delete mode 100644 .github/workflows/triageBugs.yml diff --git a/.github/ISSUE_TEMPLATE/bug.md b/.github/ISSUE_TEMPLATE/bug.md deleted file mode 100644 index 190104a47..000000000 --- a/.github/ISSUE_TEMPLATE/bug.md +++ /dev/null @@ -1,56 +0,0 @@ ---- -name: "Bug Report" -about: "Create a report to help us improve" -labels: type:bug ---- - - - -### Description - -[Summarize your problem.] - -### System Setup - -* Operating system and version: [Windows/macOS/Linux + Version ( + Desktop Environment, if Linux)] -* Cryptomator version: [Shown in the settings] -* Volume type: [Dokany/FUSE/WebDAV, shown in the settings] - -### Steps to Reproduce - -1. [First step] -2. [Second step] -3. [and so on…] - -#### Expected Behavior - -[What you expect to happen.] - -#### Actual Behavior - -[What actually happens.] - -#### Reproducibility - -[Always/Intermittent/Only once] - -### Additional Information - -[Any additional information, log files, screenshots, configuration, or data that might be necessary to reproduce the issue.] - - diff --git a/.github/ISSUE_TEMPLATE/bug.yml b/.github/ISSUE_TEMPLATE/bug.yml new file mode 100644 index 000000000..70bf9654f --- /dev/null +++ b/.github/ISSUE_TEMPLATE/bug.yml @@ -0,0 +1,93 @@ +name: Bug Report +description: Create a report to help us improve +labels: ["type:bug"] +body: + - type: checkboxes + id: terms + attributes: + label: Please agree to the following + options: + - label: I have searched [existing issues](https://github.com/cryptomator/cryptomator/issues?q=) for duplicates + required: true + - label: I agree to follow this project's [Code of Conduct](https://github.com/cryptomator/cryptomator/blob/develop/.github/CODE_OF_CONDUCT.md) + required: true + - type: input + id: summary + attributes: + label: Summary + placeholder: Please summarize your problem. + validations: + required: true + - type: textarea + id: software-versions + attributes: + label: What software is involved? + description: | + Examples: + - Operating System: Windows 10 + - Cryptomator: 1.5.16 + - LibreOffice: 7.1.4 + value: | + - Operating System: + - Cryptomator: + - … + validations: + required: true + - type: dropdown + id: volume-type + attributes: + label: Volume Type + description: What is selected under Settings → Virtual Drive? + multiple: true + options: + - FUSE + - Dokany + - WebDAV + validations: + required: false + - type: textarea + id: reproduction-steps + attributes: + label: Steps to Reproduce + value: | + 1. [First Step] + 2. [Second Step] + 3. … + validations: + required: true + - type: textarea + id: expected-behaviour + attributes: + label: Expected Behavior + placeholder: What you expect to happen. + validations: + required: true + - type: textarea + id: actual-behaviour + attributes: + label: Actual Behavior + placeholder: What actually happens. + validations: + required: true + - type: dropdown + id: reproducibility + attributes: + label: Reproducibility + description: How often does the described behaviour occur? + options: + - Always + - Intermittent + - Only once + validations: + required: true + - type: textarea + id: logs + attributes: + label: Relevant Log Output + description: Please copy and paste any relevant log output. This will be automatically formatted into code, so no need for backticks. + render: shell + - type: textarea + id: further-info + attributes: + label: Anything else? + description: Links? References? Screenshots? Configurations? Any data that might be necessary to reproduce the issue? \ No newline at end of file diff --git a/.github/ISSUE_TEMPLATE/feature.md b/.github/ISSUE_TEMPLATE/feature.md deleted file mode 100644 index 366f7ecca..000000000 --- a/.github/ISSUE_TEMPLATE/feature.md +++ /dev/null @@ -1,27 +0,0 @@ ---- -name: "Feature Request" -about: "Suggest an idea for this project" -labels: type:feature-request ---- - - - -### Summary - -[One paragraph explanation of the feature.] - -### Motivation - -[Why are we doing this? What use cases does it support? What is the expected outcome?] - -### Considered Alternatives - -[A clear and concise description of the alternative solutions you've considered.] - -### Additional Context - -[Add any other context or screenshots about the feature request here.] diff --git a/.github/ISSUE_TEMPLATE/feature.yml b/.github/ISSUE_TEMPLATE/feature.yml new file mode 100644 index 000000000..652f27234 --- /dev/null +++ b/.github/ISSUE_TEMPLATE/feature.yml @@ -0,0 +1,37 @@ +name: Feature Request +description: Suggest an idea for this project +labels: ["type:feature-request"] +body: + - type: checkboxes + id: terms + attributes: + label: Please agree to the following + options: + - label: I have searched [existing issues](https://github.com/cryptomator/cryptomator/issues?q=) for duplicates + required: true + - label: I agree to follow this project's [Code of Conduct](https://github.com/cryptomator/cryptomator/blob/develop/.github/CODE_OF_CONDUCT.md) + required: true + - type: input + id: summary + attributes: + label: Summary + placeholder: Please summarize your feature request. + validations: + required: true + - type: textarea + id: motivation + attributes: + label: Motivation + description: Who requires this feature? What problem does the user face? How would this feature solve the problem? + validations: + required: true + - type: textarea + id: alternatives + attributes: + label: Considered Alternatives + description: What current alternatives or workarounds have you considered? Is there a different way to solve the same problem? + - type: textarea + id: context + attributes: + label: Anything else? + description: Any context, suggestions, screenshots, or concepts you want to share? \ No newline at end of file diff --git a/.github/workflows/triageBugs.yml b/.github/workflows/triageBugs.yml deleted file mode 100644 index bcb0695b0..000000000 --- a/.github/workflows/triageBugs.yml +++ /dev/null @@ -1,37 +0,0 @@ -name: Bug Report Triage - -on: - issues: - types: [opened] - -jobs: - closeTemplateViolation: - name: Validate bug report against issue template - runs-on: ubuntu-latest - if: contains(github.event.issue.labels.*.name, 'type:bug') - steps: - - name: Check "Description" - if: | - !contains(github.event.issue.body, env.MUST_CONTAIN) - || contains(toJson(github.event.issue.body), env.MUST_NOT_CONTAIN) - run: exit 1 - env: - MUST_CONTAIN: '### Description' - MUST_NOT_CONTAIN: '### Description\r\n\r\n[Summarize your problem.]\r\n\r\n### System Setup' - - name: Check "Steps to Reproduce" - if: | - !contains(github.event.issue.body, env.MUST_CONTAIN) - || contains(toJson(github.event.issue.body), env.MUST_NOT_CONTAIN) - run: exit 1 - env: - MUST_CONTAIN: '### Steps to Reproduce' - MUST_NOT_CONTAIN: '### Steps to Reproduce\r\n\r\n1. [First step]\r\n2. [Second step]\r\n3. [and so on…]\r\n\r\n#### Expected Behavior' - - name: Close issue if one of the checks failed - if: ${{ failure() }} - uses: peter-evans/close-issue@v1 - with: - comment: | - This bug report did ignore our issue template. 😞 - Auto-closing this issue, since it is most likely not useful. - - _This decision was made by a bot. If you think the bot is wrong, let us know and we'll reopen this issue._ From 0db2aac7b5ee57f237744d4c014439de4ad4c19d Mon Sep 17 00:00:00 2001 From: Sebastian Stenzel Date: Fri, 16 Jul 2021 10:14:41 +0200 Subject: [PATCH 2/5] no need for wysiwyg in this plain list [ci skip] --- .github/ISSUE_TEMPLATE/bug.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/ISSUE_TEMPLATE/bug.yml b/.github/ISSUE_TEMPLATE/bug.yml index 70bf9654f..9caf4078f 100644 --- a/.github/ISSUE_TEMPLATE/bug.yml +++ b/.github/ISSUE_TEMPLATE/bug.yml @@ -31,6 +31,7 @@ body: - Operating System: - Cryptomator: - … + render: markdown validations: required: true - type: dropdown From 1ccce71abcc83b9b610916bb18f2f8738b0c04b2 Mon Sep 17 00:00:00 2001 From: Sebastian Stenzel Date: Fri, 16 Jul 2021 12:43:01 +0200 Subject: [PATCH 3/5] Revert "no need for wysiwyg in this plain list" This reverts commit 0db2aac7b5ee57f237744d4c014439de4ad4c19d. --- .github/ISSUE_TEMPLATE/bug.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/ISSUE_TEMPLATE/bug.yml b/.github/ISSUE_TEMPLATE/bug.yml index 9caf4078f..70bf9654f 100644 --- a/.github/ISSUE_TEMPLATE/bug.yml +++ b/.github/ISSUE_TEMPLATE/bug.yml @@ -31,7 +31,6 @@ body: - Operating System: - Cryptomator: - … - render: markdown validations: required: true - type: dropdown From 8d9dc3bd90ff774b70204eddb920495b36a97d6c Mon Sep 17 00:00:00 2001 From: Tobias Hagemann Date: Sat, 17 Jul 2021 12:35:50 +0200 Subject: [PATCH 4/5] Update README.md [ci skip] --- README.md | 8 -------- 1 file changed, 8 deletions(-) diff --git a/README.md b/README.md index 5e847208c..cfccf87c1 100644 --- a/README.md +++ b/README.md @@ -28,14 +28,6 @@ Cryptomator is provided free of charge as an open-source project despite the hig ### Silver Sponsors - - - - - - -
TheBestVPN
- - [Jameson Lopp](https://www.lopp.net/) --- From 397061a6e72349d876cd88734833bb05c2386340 Mon Sep 17 00:00:00 2001 From: Sebastian Stenzel Date: Mon, 19 Jul 2021 15:52:40 +0200 Subject: [PATCH 5/5] dependency update --- pom.xml | 14 +++++----- .../ui/health/HealthCheckTask.java | 10 +++---- .../cryptomator/ui/health/ReportWriter.java | 6 ++-- src/main/resources/license/THIRD-PARTY.txt | 28 +++++++++---------- 4 files changed, 29 insertions(+), 29 deletions(-) diff --git a/pom.xml b/pom.xml index e6ec12417..16411e337 100644 --- a/pom.xml +++ b/pom.xml @@ -24,23 +24,23 @@ 16 - 2.1.0-beta8 + 2.1.0-beta9 1.0.0-rc1 1.0.0-beta2 1.0.0-beta2 1.0.0-beta1 1.3.1 1.3.1 - 1.2.4 + 1.2.5 16 3.12.0 - 3.17.0 + 3.18.1 2.2 30.1.1-jre 2.37 - 2.8.6 + 2.8.7 1.7.31 1.2.3 @@ -190,7 +190,7 @@ com.google.jimfs jimfs - 1.1 + 1.2 test @@ -236,12 +236,12 @@ org.jacoco jacoco-maven-plugin - 0.8.6 + 0.8.7 org.owasp dependency-check-maven - 6.0.3 + 6.2.2 diff --git a/src/main/java/org/cryptomator/ui/health/HealthCheckTask.java b/src/main/java/org/cryptomator/ui/health/HealthCheckTask.java index 3efe333b3..57cf6eb62 100644 --- a/src/main/java/org/cryptomator/ui/health/HealthCheckTask.java +++ b/src/main/java/org/cryptomator/ui/health/HealthCheckTask.java @@ -47,10 +47,10 @@ class HealthCheckTask extends Task { this.check = Objects.requireNonNull(check); this.results = FXCollections.observableArrayList(Result::observables); try { - updateTitle(resourceBundle.getString("health." + check.identifier())); + updateTitle(resourceBundle.getString("health." + check.name())); } catch (MissingResourceException e) { - LOG.warn("Missing proper name for health check {}, falling back to default.", check.identifier()); - updateTitle(check.identifier()); + LOG.warn("Missing proper name for health check {}, falling back to default.", check.name()); + updateTitle(check.name()); } this.durationInMillis = new SimpleLongProperty(-1); this.chosenForExecution = new SimpleBooleanProperty(); @@ -74,12 +74,12 @@ class HealthCheckTask extends Task { @Override protected void scheduled() { - LOG.info("starting {}", check.identifier()); + LOG.info("starting {}", check.name()); } @Override protected void done() { - LOG.info("finished {}", check.identifier()); + LOG.info("finished {}", check.name()); } /* Getter */ diff --git a/src/main/java/org/cryptomator/ui/health/ReportWriter.java b/src/main/java/org/cryptomator/ui/health/ReportWriter.java index 6039901ef..4366d14f0 100644 --- a/src/main/java/org/cryptomator/ui/health/ReportWriter.java +++ b/src/main/java/org/cryptomator/ui/health/ReportWriter.java @@ -64,10 +64,10 @@ public class ReportWriter { writer.write(REPORT_HEADER.formatted(vaultConfig.getId(), vault.getDisplayName(), vault.getPath())); for (var task : tasks) { if (task.getState() == Worker.State.READY) { - LOG.debug("Skipping not performed check {}.", task.getCheck().identifier()); + LOG.debug("Skipping not performed check {}.", task.getCheck().name()); continue; } - writer.write(REPORT_CHECK_HEADER.formatted(task.getCheck().identifier())); + writer.write(REPORT_CHECK_HEADER.formatted(task.getCheck().name())); switch (task.getState()) { case SUCCEEDED -> { writer.write("STATUS: SUCCESS\nRESULTS:\n"); @@ -77,7 +77,7 @@ public class ReportWriter { } case CANCELLED -> writer.write("STATUS: CANCELED\n"); case FAILED -> { - writer.write("STATUS: FAILED\nREASON:\n" + task.getCheck().identifier()); + writer.write("STATUS: FAILED\nREASON:\n" + task.getCheck().name()); writer.write(prepareFailureMsg(task)); } case RUNNING, SCHEDULED -> throw new IllegalStateException("Checks are still running."); diff --git a/src/main/resources/license/THIRD-PARTY.txt b/src/main/resources/license/THIRD-PARTY.txt index 2d234f01c..6c58b1eee 100644 --- a/src/main/resources/license/THIRD-PARTY.txt +++ b/src/main/resources/license/THIRD-PARTY.txt @@ -18,7 +18,7 @@ Cryptomator uses 45 third-party dependencies under the following licenses: - jnr-constants (com.github.jnr:jnr-constants:0.9.15 - http://github.com/jnr/jnr-constants) - jnr-ffi (com.github.jnr:jnr-ffi:2.1.12 - http://github.com/jnr/jnr-ffi) - FindBugs-jsr305 (com.google.code.findbugs:jsr305:3.0.2 - http://findbugs.sourceforge.net/) - - Gson (com.google.code.gson:gson:2.8.6 - https://github.com/google/gson/gson) + - Gson (com.google.code.gson:gson:2.8.7 - https://github.com/google/gson/gson) - Dagger (com.google.dagger:dagger:2.37 - https://github.com/google/dagger) - error-prone annotations (com.google.errorprone:error_prone_annotations:2.5.1 - http://nexus.sonatype.org/oss-repository-hosting.html/error_prone_parent/error_prone_annotations) - Guava InternalFutureFailureAccess and InternalFutures (com.google.guava:failureaccess:1.0.1 - https://github.com/google/guava/failureaccess) @@ -32,12 +32,12 @@ Cryptomator uses 45 third-party dependencies under the following licenses: - Apache Commons Lang (org.apache.commons:commons-lang3:3.12.0 - https://commons.apache.org/proper/commons-lang/) - Apache HttpCore (org.apache.httpcomponents:httpcore:4.4.14 - http://hc.apache.org/httpcomponents-core-ga) - Jackrabbit WebDAV Library (org.apache.jackrabbit:jackrabbit-webdav:2.21.5 - http://jackrabbit.apache.org/jackrabbit-webdav/) - - Jetty :: Http Utility (org.eclipse.jetty:jetty-http:10.0.3 - https://eclipse.org/jetty/jetty-http) - - Jetty :: IO Utility (org.eclipse.jetty:jetty-io:10.0.3 - https://eclipse.org/jetty/jetty-io) - - Jetty :: Security (org.eclipse.jetty:jetty-security:10.0.3 - https://eclipse.org/jetty/jetty-security) - - Jetty :: Server Core (org.eclipse.jetty:jetty-server:10.0.3 - https://eclipse.org/jetty/jetty-server) - - Jetty :: Servlet Handling (org.eclipse.jetty:jetty-servlet:10.0.3 - https://eclipse.org/jetty/jetty-servlet) - - Jetty :: Utilities (org.eclipse.jetty:jetty-util:10.0.3 - https://eclipse.org/jetty/jetty-util) + - Jetty :: Http Utility (org.eclipse.jetty:jetty-http:10.0.6 - https://eclipse.org/jetty/jetty-http) + - Jetty :: IO Utility (org.eclipse.jetty:jetty-io:10.0.6 - https://eclipse.org/jetty/jetty-io) + - Jetty :: Security (org.eclipse.jetty:jetty-security:10.0.6 - https://eclipse.org/jetty/jetty-security) + - Jetty :: Server Core (org.eclipse.jetty:jetty-server:10.0.6 - https://eclipse.org/jetty/jetty-server) + - Jetty :: Servlet Handling (org.eclipse.jetty:jetty-servlet:10.0.6 - https://eclipse.org/jetty/jetty-servlet) + - Jetty :: Utilities (org.eclipse.jetty:jetty-util:10.0.6 - https://eclipse.org/jetty/jetty-util) - Jetty :: Servlet API and Schemas for JPMS and OSGi (org.eclipse.jetty.toolchain:jetty-servlet-api:4.0.6 - https://eclipse.org/jetty/jetty-servlet-api) BSD: - asm (org.ow2.asm:asm:7.1 - http://asm.ow2.org/) @@ -48,12 +48,12 @@ Cryptomator uses 45 third-party dependencies under the following licenses: Eclipse Public License - Version 1.0: - Jetty :: Servlet API and Schemas for JPMS and OSGi (org.eclipse.jetty.toolchain:jetty-servlet-api:4.0.6 - https://eclipse.org/jetty/jetty-servlet-api) Eclipse Public License - Version 2.0: - - Jetty :: Http Utility (org.eclipse.jetty:jetty-http:10.0.3 - https://eclipse.org/jetty/jetty-http) - - Jetty :: IO Utility (org.eclipse.jetty:jetty-io:10.0.3 - https://eclipse.org/jetty/jetty-io) - - Jetty :: Security (org.eclipse.jetty:jetty-security:10.0.3 - https://eclipse.org/jetty/jetty-security) - - Jetty :: Server Core (org.eclipse.jetty:jetty-server:10.0.3 - https://eclipse.org/jetty/jetty-server) - - Jetty :: Servlet Handling (org.eclipse.jetty:jetty-servlet:10.0.3 - https://eclipse.org/jetty/jetty-servlet) - - Jetty :: Utilities (org.eclipse.jetty:jetty-util:10.0.3 - https://eclipse.org/jetty/jetty-util) + - Jetty :: Http Utility (org.eclipse.jetty:jetty-http:10.0.6 - https://eclipse.org/jetty/jetty-http) + - Jetty :: IO Utility (org.eclipse.jetty:jetty-io:10.0.6 - https://eclipse.org/jetty/jetty-io) + - Jetty :: Security (org.eclipse.jetty:jetty-security:10.0.6 - https://eclipse.org/jetty/jetty-security) + - Jetty :: Server Core (org.eclipse.jetty:jetty-server:10.0.6 - https://eclipse.org/jetty/jetty-server) + - Jetty :: Servlet Handling (org.eclipse.jetty:jetty-servlet:10.0.6 - https://eclipse.org/jetty/jetty-servlet) + - Jetty :: Utilities (org.eclipse.jetty:jetty-util:10.0.6 - https://eclipse.org/jetty/jetty-util) Eclipse Public License - v 1.0: - Logback Classic Module (ch.qos.logback:logback-classic:1.2.3 - http://logback.qos.ch/logback-classic) - Logback Core Module (ch.qos.logback:logback-core:1.2.3 - http://logback.qos.ch/logback-core) @@ -74,7 +74,7 @@ Cryptomator uses 45 third-party dependencies under the following licenses: - Java Native Access (net.java.dev.jna:jna:5.7.0 - https://github.com/java-native-access/jna) - Java Native Access Platform (net.java.dev.jna:jna-platform:5.7.0 - https://github.com/java-native-access/jna) MIT License: - - java jwt (com.auth0:java-jwt:3.17.0 - https://github.com/auth0/java-jwt) + - java jwt (com.auth0:java-jwt:3.18.1 - https://github.com/auth0/java-jwt) - jnr-x86asm (com.github.jnr:jnr-x86asm:1.0.2 - http://github.com/jnr/jnr-x86asm) - jnr-fuse (com.github.serceman:jnr-fuse:0.5.5 - https://github.com/SerCeMan/jnr-fuse) - zxcvbn4j (com.nulab-inc:zxcvbn:1.3.0 - https://github.com/nulab/zxcvbn4j)