Skip to content

Validate url before visiting

Savas Vedova requested to merge check-url-security into master

What does this MR do and why?

Checks whether the visited URL is safe or not before visiting. If it's not, the function now throws an error.

Screenshots or screen recordings

Nothing changes visually.

How to set up and validate locally

This is mostly a change that won't affect the functionality. But to test it you can:

  1. Run tests locally
  2. Perform an operation that uses visitUrl (for instance, if you create an issue from a finding in the Mr security reports it'll use visitUrl to take you to the issue page).

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports