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

Template logs are their corresponding AppVMs #476

Closed
emkll opened this issue Mar 2, 2020 · 2 comments · Fixed by #487
Closed

Template logs are their corresponding AppVMs #476

emkll opened this issue Mar 2, 2020 · 2 comments · Fixed by #487

Comments

@emkll
Copy link
Contributor

emkll commented Mar 2, 2020

This will make debugging updates more difficult, and will make it difficult to distinguish.

Ideally, we would want to detect the VM based on the qrexec call. @kushaldas , is there a way we can detect the source of the ? Otherwise a compromised VM could spoof log source.

If we cannot detect source VM based on qrexec call, we should consider the following:
Replace vmname context in the configuration step to the that of the template [1] the and for AppVMs use the strategy used in sd-whonix via rc.local[2]

[1]

sd-rsyslog-for-sd-devices:

[2]

@kushaldas
Copy link
Contributor

We should follow the [2] like sd-whonix in all VMs.

@conorsch
Copy link
Contributor

conorsch commented Mar 2, 2020

Another option is to use the hostname reported by the logclient machine, e.g. via hostname, rather than hardcoding the client name in a config file. Doing so would clearly distinguish between Template & AppVMs, as well as allow us to delete a considerable amount of configuration logic. Example of inferring hostname: https://github.com/freedomofpress/securedrop-log/compare/infer-hostname

The problem of "a compromised AppVM can claim to be a differently named VM" is a separate issue, and one I don't think we should try hard to solve for the pilot, where logging will be most useful for debugging application components. When we try to resolve the masquerade problem, we should consider using dom0 to aggregate and store the logs, since dom0 already stores all syslog for each VM, with unique names for Template & AppVMs. See related comment #463 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants