Skip to content
This repository has been archived by the owner on Nov 4, 2024. It is now read-only.

Remove SSH observatory. #235

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

floatingatoll
Copy link

Fixes #234. This seems to work correctly for me in local testing, and the webpack build looks correct.

@@ -21,7 +21,6 @@ module.exports = {
title: 'Mozilla Observatory',
urls: {
api: 'https://http-observatory.security.mozilla.org/api/v1/',
ssh: 'https://sshscan.rubidus.com/api/v1/',
Copy link
Member

Choose a reason for hiding this comment

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

This makes me wonder: can we link to some other SSH scan tool for people who have been using Observatory for SSH scanning?

Copy link
Member

Choose a reason for hiding this comment

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

We could link users directly to the CLI tool here? https://github.com/mozilla/ssh_scan

Copy link
Member

@groovecoder groovecoder left a comment

Choose a reason for hiding this comment

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

Code LGTM and works fine on my local machine. Made 1 comment that could be a follow-up if we think we can link to some 3rd-party SSH scanner for people who were using observatory for that in the past?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Decomission the SSH Tab
2 participants