Skip to content

Conversation

corbob
Copy link
Member

@corbob corbob commented Oct 1, 2025

Description Of Changes

Remove link from the list of known proxy issues.

Motivation and Context

The issue linked as a known proxy issue is not actually an issue. It was created due to a misunderstanding of how the VirusTotal check was working, and ultimately the VirusTotal check is using the proxy settings.

Testing

  • I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

  • Minor documentation fix (typos etc.).
  • Major documentation change (refactoring, reformatting or adding documentation to existing page).
  • New documentation page added.
  • The change I have made should have a video added, and I have raised an issue for this.
    • Issue #

Change Checklist

  • Requires a change to menu structure (top or left-hand side)/
  • Menu structure has been updated
  • Images added to the img repository?
    • PR -

Related Issue

N/A

The issue linked as a known proxy issue is not actually an issue. It was
created due to a misunderstanding of how the VirusTotal check was
working, and ultimately the VirusTotal check is using the proxy settings.
@gep13
Copy link
Member

gep13 commented Oct 2, 2025

Should the linked issue be closed? Or left open?

If it was created based on a misunderstanding, it would be great to capture some of those details in the issue, and then close it out.

Once the issue is closed, we could then get this PR merged.

@gep13
Copy link
Member

gep13 commented Oct 2, 2025

@corbob should have tagged you on the above ☝🏻

@corbob
Copy link
Member Author

corbob commented Oct 2, 2025

Good point @gep13. The linked internal issue is closed, but it looks like Unito didn't close the public one. I've pinged Paul on the integration, and will wait for him before I close the public issue.

With updating the issue with the misunderstanding, I have an idea, and have added it to our discussion meeting when I return from holidays. As such, I'll convert this PR to draft for now as I have some ideas that might lead to additional related changes here.

@corbob corbob marked this pull request as draft October 2, 2025 17:38
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