Include user's command in smart server header

Bug #309540 reported by Jonathan Lange
2
Affects Status Importance Assigned to Milestone
Bazaar
Confirmed
Wishlist
Unassigned

Bug Description

There's no good way for a Bazaar host to log information about what its users think that they are doing.

One way to make this easier would be to send the command that the user supplied to bzr ('info', 'log' etc) in the smart server hello command, or something similar.

Revision history for this message
Andrew Bennetts (spiv) wrote :

The "hello" command is being phased out; recent clients shouldn't be using it much, if at all.

It might be reasonable to add a header like "CLI command": "push" to requests, though. (But I wouldn't want to send more detail than just the command name, and even then just command names from core and not plugins, to minimise chances of inadvertently leaking data that the client doesn't want to share.)

Revision history for this message
John A Meinel (jameinel) wrote :

We have '-Dhpss' on the server side now, though that logs the specific actions, and not the user requests. (So it says that they are fetching data, but not whether that is 'log' vs 'pull' vs 'branch'.

Changed in bzr:
importance: Undecided → Wishlist
status: New → Confirmed
Jelmer Vernooij (jelmer)
tags: added: check-for-breezy
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.