culturesofknowledge / emlo-project

Repository to manage the upgrade, development and maintenance of EMLO
0 stars 0 forks source link

Add session user to audit log #27

Open anusharanganathan opened 2 years ago

philipkcl commented 1 year ago

@anusharanganathan to be discuss this one

J4bbi commented 1 year ago

@philipkcl , this is done?

philipkcl commented 1 year ago

@J4bbi, I need more information for this one, do you have any idea ?

J4bbi commented 1 year ago

The problem this issue fails to describe is that in the old system the Postgres user would be logged as the author of a change. In our system we want to write the Django user into the audit log. So the problem is, how to reconcile these two, different concepts of users.

If I remember correctly, we will still keep all the database audit triggers, and if so, does a signal on the audit table intercept the new entry and replace the Postgres user with the Django username?

philipkcl commented 1 year ago

your remember is correct,

but how important to "reconcile these two"?

the username field in audit should be for display only, no other features right?

J4bbi commented 1 year ago

the username field in audit should be for display only, no other features right?

I'm not sure, I think so

philipkcl commented 1 year ago

system seem showing ''__unknown_user" after using uploader. to be study that problem