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

Notification warning count inconsistency #859

Open
grke opened this issue Apr 19, 2020 · 1 comment
Open

Notification warning count inconsistency #859

grke opened this issue Apr 19, 2020 · 1 comment

Comments

@grke
Copy link
Owner

grke commented Apr 19, 2020

Reported on the mailing list, burp 2.2.18

When I try to restore the backup into a temporary directory
/media/backup/snapshot by the command:
"burp -c /etc/burp/ubuntu.conf -a r -d /media/backup/snapshot",
everything works fine except for an unknown warning returned by burp with exit
code 2.

The notification email subject is "restore succeeded: ubuntu (1 warnings)" but the counters in the log message say 0 warnings.

In the server-side log:

burp[27654]: doing restore confirmed
burp[27655]: Logging to /backup/data/ubuntu/0000037 2020-04-16 20:06:10
+0200/restorelog
burp[27654]: WARNING: /media/backup/snapshot: Is not in a subdir of
'/media/backup/snapshot'
burp[27655]: Compressing /backup/data/ubuntu/0000037 2020-04-16
20:06:10 +0200/restorelog to /backup/data/ubuntu/0000037 2020-04-16 20:06:10
+0200/restorelog.gz...
burp[27655]: WARNING: /media/backup/snapshot: Is not in a subdir of
'/media/backup/snapshot'
burp[27654]: restore finished
@grke
Copy link
Owner Author

grke commented Apr 19, 2020

It looks to me like maybe the canonicalisation check gets confused when dealing with the final directory, when specifying an identical destination directory.

And also, there may be some kind of race condition where a very late warning gets logged server-side after the server-side client-specific log gets closed.

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

1 participant