Skip to content

Remove double slashes from urls generated by PseudonymizationHelper

Niko Belokolodov requested to merge 511-remove-double-slash-from-urls into master

What does this MR do and why?

Investigation: https://gitlab.com/gitlab-org/product-intelligence/-/issues/509 Please see https://gitlab.com/gitlab-org/product-intelligence/-/issues/509#note_829341428 for details.

We currently submit some URLs in snowplow as page_url with two slashes, instead of one. e.g. //-/security/vulnerabilities instead of /-/security/vulnerabilities. We want to fix this as it makes it's not correct and it could hinder some data analysis.

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

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

Related to #511 (closed)

Merge request reports