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

Address several UX issues #65

Merged
merged 5 commits into from
Mar 16, 2023
Merged

Address several UX issues #65

merged 5 commits into from
Mar 16, 2023

Conversation

dylan-bourque
Copy link

This PR addresses several issues found while using Perseus internally at CrowdStrike.

  • The client CLI was not able to connect to a remove server using TLS.
    • This was addressed by defaulting to using TLS when dialing and adding a new --insecure flag to the query and update commands to manually disable TLS where necessary (such as when doing local debugging)
  • The module details page in the web UI did not show any data.
    • There was a change to the structure of the response from the .../module-versions endpoint and the UI was not updated
  • query list-modules would fail if a module has no stable versions in the database
    • Added logic to attempt to find a pre-release if no stable release exists

Dylan Bourque added 4 commits March 15, 2023 14:48
update the JS code to sync with changes in the response returned by `.../module-versions`
corrected attribute generation code in `debugLog()` to work correctly when >1 k/v pair is provided
update gRPC client dial options to use TLS by default
added new `--insecure` CLI flag to explicitly disable TLS
…o stable versions

updated `query list-modules` to try to find a pre-release version if no stable version exists for a module
@dylan-bourque dylan-bourque requested review from bndw and a team March 15, 2023 20:11
internal/server/web/js/data.js Outdated Show resolved Hide resolved
@dylan-bourque dylan-bourque merged commit de1907b into main Mar 16, 2023
@dylan-bourque dylan-bourque deleted the fix/ux-bugs branch March 16, 2023 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants