Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Manage the HTTPS certificate from the menu and ask Safari users to install it at startup #941

Merged
merged 19 commits into from
May 8, 2024

Conversation

MatteoPologruto
Copy link
Contributor

@MatteoPologruto MatteoPologruto commented Apr 22, 2024

Please check if the PR fulfills these requirements

  • The PR has no duplicates (please search among the Pull Requests
    before creating one)
  • Tests for the changes have been added (for bug fixes / features)
  • What kind of change does this PR introduce?

Infrastructure enhancement

  • What is the new behavior?

Safari users are prompted at startup to install the HTTPS certificate and their preference is saved in the config file.
It is now possible to obtain the certificate informations (installed, trusted and expiration date) from the Agent menu before performing any action on the keychain.
If the installed certificate is expiring in less than one month, the Agent will ask the user to update it.

  • Does this PR introduce a breaking change?

No

  • Other information:

@MatteoPologruto MatteoPologruto added type: enhancement Proposed improvement topic: code Related to content of the project itself labels Apr 22, 2024
@MatteoPologruto MatteoPologruto self-assigned this Apr 22, 2024
@codecov-commenter
Copy link

codecov-commenter commented Apr 22, 2024

Codecov Report

Attention: Patch coverage is 0% with 110 lines in your changes are missing coverage. Please review.

Project coverage is 20.72%. Comparing base (2d6395d) to head (3487304).

Files Patch % Lines
main.go 0.00% 32 Missing ⚠️
systray/systray_real.go 0.00% 29 Missing ⚠️
certificates/certificates.go 0.00% 28 Missing ⚠️
config/config.go 0.00% 11 Missing ⚠️
certificates/install_default.go 0.00% 6 Missing ⚠️
utilities/utilities.go 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #941      +/-   ##
==========================================
- Coverage   21.42%   20.72%   -0.71%     
==========================================
  Files          42       42              
  Lines        2506     2591      +85     
==========================================
  Hits          537      537              
- Misses       1872     1957      +85     
  Partials       97       97              
Flag Coverage Δ
unit 20.72% <0.00%> (-0.71%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MatteoPologruto MatteoPologruto force-pushed the check-valid-certificate branch 12 times, most recently from 2b8b5d4 to b3aa9ab Compare April 24, 2024 15:37
@MatteoPologruto MatteoPologruto force-pushed the check-valid-certificate branch 14 times, most recently from 4bbef3c to 8a020e9 Compare May 7, 2024 07:21
certificates/certificates.go Outdated Show resolved Hide resolved
systray/systray_real.go Show resolved Hide resolved
certificates/install_darwin.go Show resolved Hide resolved
Copy link
Contributor

@umbynos umbynos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall Code LGTM 🏁

main.go Outdated Show resolved Hide resolved
certificates/install_darwin.go Show resolved Hide resolved
@MatteoPologruto MatteoPologruto merged commit 222a505 into arduino:main May 8, 2024
20 checks passed
@MatteoPologruto MatteoPologruto deleted the check-valid-certificate branch May 8, 2024 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement
Projects
None yet
4 participants