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.

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

"Syntax error in revision argument" when accessing deleted file #191

Open
danielmarschall opened this issue Sep 30, 2023 · 2 comments
Open

Comments

@danielmarschall
Copy link
Contributor

I noticed that in my php_error.log there are a lot of error messages:

[30-Sep-2023 22:59:35 europe/berlin] Fehler beim Ausführen des Befehls: /usr/bin/svn --non-interactive --config-dir /tmp/websvn log --xml --quiet --limit 2 -r -1:1 'file:///daten/subversion/distributed/ViaThinkSoft%20Distributed/src/de/viathinksoft/distributed/apps/immortal/iterator/marschall/MarschallSectionIterator.java@HEAD'
[30-Sep-2023 22:59:35 europe/berlin] No output on STDOUT.E205000: Syntax error in revision argument '-1:1'

The URL which was accessed and caused the error is this:

https://......./websvn/filedetails.php?path=%2FViaThinkSoft%20Distributed%2Fsrc%2Fde%2Fviathinksoft%2Fdistributed%2Fapps%2Fimmortal%2Fiterator%2Fmarschall%2FMarschallSectionIterator.java&repname=distributed&rev=-1

The directory `.../src/de/viathinksoft/distributed/..." was deleted some time ago.

The GUI correctly shows "404". However, the php_error.log still contains the output due to a failed internal shell access.

I am not sure why this URL was accessed at all. It could be a search engine which tries to open an old URL. Or is it possible that some parts of the WebSVN links to this file, e.g. if you look in old Logs/Diffs?

@michael-o
Copy link
Member

Well, yes, -1 should not be passed at all. I'll try to look at it next month.

@michael-o michael-o self-assigned this Sep 30, 2023
@michael-o
Copy link
Member

I cannot see any spot in the code where the -1 is set. I was able to force the issue by supplying -1 only manually. It'd be could if could try to reproduce it.

@michael-o michael-o removed their assignment Jan 26, 2024
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

No branches or pull requests

2 participants