Skip to content

Update encoding method for logs_tree request

Jacques Erasmus requested to merge use-correct-encoding-method-logs-tree into master

What does this MR do and why?

Some users have reported that commit data is not being loaded for directories more than two levels deep.

It turns out the frontend was encoding / in the logs_tree URI.

Issue: #385699 (closed)

Screenshots or screen recordings

No visual changes are expected

Before After
/ characters are encoded / characters are not encoded
Screenshot_2023-11-08_at_14.58.32 Screenshot_2023-11-08_at_14.59.09

How to set up and validate locally

  1. Open a directory more than 2 levels deep
  2. Inspect the network requests
  3. The request to logs_tree should not have encoded / characters

MR acceptance checklist

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

Edited by Jacques Erasmus

Merge request reports