Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

File paths missing slash in the Snitch table #54

Open
pedro-mendonca opened this issue Nov 23, 2023 · 2 comments · May be fixed by #53
Open

File paths missing slash in the Snitch table #54

pedro-mendonca opened this issue Nov 23, 2023 · 2 comments · May be fixed by #53
Labels
bug

Comments

@pedro-mendonca
Copy link
Contributor

Describe the bug
The file path in the connections have no slashes.

To Reproduce
Steps to reproduce the behavior:

  1. Go to Snitch page
  2. Check the file paths in the 'File' column, all have this wrong pattern:

imagem

Expected behavior
The file paths are unreadable, should be kept the slashes as the pattern below:

imagem
@pedro-mendonca pedro-mendonca linked a pull request Nov 23, 2023 that will close this issue
Open
@stklcode
Copy link
Contributor

stklcode commented Nov 25, 2023 •
edited
Loading

Interesting... I am unable to reproduce this issue, my paths look just as expected within a test container and on a real-world site.

snitch_urls

From the backslashes in your expected screenshot, I'd guess that you are running your WP site on a Windows machine, right?

Also notice that "my" paths are relativ wo the WP root, yours seem to be absolute or at least starting from some levels above WP 🤔 We should have a deeper look into the routine, I have a suspicion that the actual bug is not missing slashes in the output.

@stklcode stklcode added the bug label Nov 25, 2023
@pedro-mendonca
Copy link
Contributor Author

Right, I'm testing on a Windows machine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Don't remove slashes in file path pedro-mendonca/snitch
2 participants
@pedro-mendonca @stklcode